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.

Sign up for GitHub

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

Jump to bottom

refactor(CL Swaps): remove redundant swap API: SwapInAmtGivenOut #4426

Closed
Tracked by #4424
p0mvn opened this issue Feb 27, 2023 · 1 comment
Closed
Tracked by #4424

refactor(CL Swaps): remove redundant swap API: SwapInAmtGivenOut #4426

p0mvn opened this issue Feb 27, 2023 · 1 comment
Assignees
@czarcas7ic
Labels
C:x/concentrated-liquidity F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board
Milestone
CL 9

Comments

@p0mvn
Copy link
Member

p0mvn commented Feb 27, 2023

Background

Similar to #4425 but for the swap in the other direction.

Acceptance Criteria

  • method removed
  • logic ported to another public API
  • tests updated
@p0mvn p0mvn added C:x/concentrated-liquidity F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board T:ice-box labels Feb 27, 2023
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Osmosis Chain Development Feb 27, 2023
@p0mvn p0mvn mentioned this issue Feb 27, 2023
Closed
@p0mvn p0mvn removed the T:ice-box label Feb 28, 2023
@p0mvn p0mvn added this to the CL 9 milestone Feb 28, 2023
@czarcas7ic czarcas7ic self-assigned this Mar 2, 2023
@czarcas7ic czarcas7ic mentioned this issue Mar 2, 2023
Closed
@p0mvn p0mvn moved this from Needs Triage 🔍 to In Progress🏃 in Osmosis Chain Development Mar 2, 2023
@czarcas7ic czarcas7ic moved this from In Progress🏃 to Blocked ❌ in Osmosis Chain Development Mar 2, 2023
@p0mvn p0mvn mentioned this issue Mar 8, 2023
Merged
2 tasks
@p0mvn
Copy link
Member Author

p0mvn commented Mar 26, 2023

Complete in: #4533

@p0mvn p0mvn closed this as completed Mar 26, 2023
@github-project-automation github-project-automation bot moved this from Blocked ❌ to Done ✅ in Osmosis Chain Development Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@czarcas7ic czarcas7ic

Labels
C:x/concentrated-liquidity F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board
Projects
Osmosis Chain Development
Archived in project
Milestone
CL 9
Development

Successfully merging a pull request may close this issue.

CL: make swapInAmtGivenOut mutate user/pool balances osmosis-labs/osmosis
2 participants
@p0mvn @czarcas7ic