Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CL Hooks]: Add hooks source and clean up source code folder #6860

Merged
merged 19 commits into from
Nov 22, 2023

Conversation

AlpinYukseloglu
Copy link
Contributor

What is the purpose of the change

This PR adds the source code used for testing in #6859. It also cleans up the testcontracts folder to be more approachable/readable.

This is done in a separate PR to avoid cluttering the base PR since the changes are light & state compatible but the diff is large

Copy link
Member

@p0mvn p0mvn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet


#[derive(Serialize, Deserialize, Clone, Debug, PartialEq, JsonSchema)]
pub struct State {
pub count: i64,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason we keep a counter for this contract?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is unused in this case but will likely be important for #6862 so I had just left it there

lower_tick: _,
upper_tick: _,
} => {
let mut response = Response::new();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems that the following lines are duplicated regardless of which message (hook) it is, can we create a method for these lines?

Base automatically changed from alpo/cl-hooks-wiring to main November 22, 2023 07:57
@AlpinYukseloglu AlpinYukseloglu added V:state/compatible/no_backport State machine compatible PR, depends on prior breaks and removed V:state/compatible/no_backport State machine compatible PR, depends on prior breaks labels Nov 22, 2023
@AlpinYukseloglu AlpinYukseloglu merged commit 73d0141 into main Nov 22, 2023
1 check passed
@AlpinYukseloglu AlpinYukseloglu deleted the alpo/cl-hooks-test-source branch November 22, 2023 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:no-changelog C:x/concentrated-liquidity V:state/compatible/no_backport State machine compatible PR, depends on prior breaks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants