-
Notifications
You must be signed in to change notification settings - Fork 589
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
refactor/test(CL): LP methods with lower min spot price #6323
Merged
Merged
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
b335df5
refactor: state-compatible big decimal tick to sqrt price conversions
p0mvn c2af17f
updates and clean ups
p0mvn 3e51683
lint
p0mvn 7db80ff
refactor: breaking CL tick to sqrt price math min spot price relaxation
p0mvn 5aec022
refactor/test(CL): CreatePosition with lower min spot price
p0mvn 7f2f1a6
clean up
p0mvn 6919feb
update GetTickLiquidityForFullRange query
p0mvn aa9c58f
fix tests
p0mvn e212343
withdraw position tests
p0mvn 0ef4a5e
add to position test
p0mvn cd339ca
updates
p0mvn 64d145e
updates
p0mvn 6116ac3
clean up
p0mvn 77494a2
Merge branch 'main' into roman/min-spot-price-lp
p0mvn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be too late for this but I wonder if it would have been beneficial to have a special TODO for precision stuff to make it easier to be sure we don't miss any of these later (e.g.
TODO (precision):
so we can search for that when #5726 is done)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that's a good idea!
However, I made sure that every single one is tracked in the epic though so not worried about misses:
#5864