-
Notifications
You must be signed in to change notification settings - Fork 607
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
feat: CreateCLPool & LinkCFMMtoCL pool into one gov-prop #5890
Conversation
bb68eef
to
957cf62
Compare
9e63479
to
94be8c7
Compare
8c933ac
to
f10f334
Compare
f18b523
to
b09b70b
Compare
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.
First pass looks good, it appears you only tested that the prop gets posted though according to your log though. Can you pass the prop and confirm the results? After that I think we can merge.
just checked and it works fine!
|
b09b70b
to
88d175e
Compare
d3bae35
to
e9ba483
Compare
merging this as the gov prop has been tested and logs are posted |
Closes: #5888
What is the purpose of the change
Currently we have 2 gov prop message to migrate pool from GAMM.
Create CL Pool
Link GammPool with CL pool
This take 2 gov period which is too long. Lets create a gov-prop that will handle both of these commands in one
Testing and Verifying
localosmosis tested
Documentation and Release Note
Unreleased
section ofCHANGELOG.md
?Where is the change documented?
x/{module}/README.md
)