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

[CL]: Provide barebones concentrated pool implementation #3039

Closed
p0mvn opened this issue Oct 18, 2022 · 1 comment
Closed

[CL]: Provide barebones concentrated pool implementation #3039

p0mvn opened this issue Oct 18, 2022 · 1 comment
Assignees
@mattverse
Labels
C:x/gamm Changes, features and bugs related to the gamm module.
Milestone
CL 1: Provide Liq...

Comments

@p0mvn
Copy link
Member

p0mvn commented Oct 18, 2022

Background

Building up on top of #3038, we need a barebones concentrated liquidity pool implementation inside pool-models.

Implement the refactored PoolI interfaces by adding a concentrated folder inside pool-models in gamm.

concentrated liquidity is planned to be a separate module. Therefore, gamm is not the best location for pool-models. We can consider move the pool-models director out of gamm in a future issue/PR.

Suggested Design

  • new barebones implementation of PoolI in pool-models

Acceptance Criteria
@p0mvn p0mvn added the C:x/gamm Changes, features and bugs related to the gamm module. label Oct 18, 2022
@p0mvn p0mvn mentioned this issue Oct 18, 2022
Closed
@p0mvn
Copy link
Member Author

p0mvn commented Oct 18, 2022

Closed by #3040

@p0mvn p0mvn closed this as completed Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mattverse mattverse

Labels
C:x/gamm Changes, features and bugs related to the gamm module.
Projects
Osmosis Chain Development
Archived in project
Milestone
CL 1: Provide Liquidity and Swap With...
Development

No branches or pull requests
2 participants
@p0mvn @mattverse