-
Notifications
You must be signed in to change notification settings - Fork 91
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/curve migration #921
Feat/curve migration #921
Conversation
Hi @rndquu i did an initial review, thank you for the comments explaining some values (e.g. ln(2) to calc time). I have not tested it yet. Do you plan to rework and merge together with UI changes? |
The plan is to prepare the mainnet deployment script so that @pavlovcik could run Right now the redeployment plan is this one:
Overall the deployment script is ready, I will run a few tests tomorrow of the deployed contracts on the forked state.
UI requires metapool to be deployed for it to work on the mainnet. So the plan is to prepare the mainnet deployment, check how it works, apply it and then update UI. |
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.
time to launch team 🚀
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.
I did an another review round and approving as well from my side.
This PR refactors mainnet migrations to eliminate manual steps like:
P.S. I've tested the migration against the anvil forked from mainnet with this test file which passed successfully