-
Notifications
You must be signed in to change notification settings - Fork 4
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
Vault deployment #58
Vault deployment #58
Conversation
naddison36
commented
Oct 20, 2022
•
edited
Loading
edited
- Added config for new vaults on mainnet
- Hardhat tasks to snap basic vault and convex meta vault details
- Hardhat task refactoring
@@ -149,11 +149,13 @@ export async function deployConvex3CrvLiquidatorVault( | |||
`Convex3CrvLiquidatorVault ${name} (${symbol})`, | |||
constructorArguments, | |||
) | |||
|
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.
is getFactoryMetapoolLinkAddresses used?
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.
no, Curve3CrvMetapoolCalculatorLibrary
and Curve3CrvFactoryMetapoolCalculatorLibrary
have the same ABI so I could get away with just using getMetapoolLinkAddresses
even through BUSD and FRAX are Curve Metapools.
The logic for picking the correct library address is in the convex-3crv-vault-deploy
task at
metavaults/tasks/convex3CrvVault.ts
Line 231 in f3e5b18
calculatorLibraryAddress = convex3CrvPool.isFactory |
@@ -52,3 +52,45 @@ subtask("curve-add", "Add liquidity to Curve 3Pool") | |||
task("curve-add").setAction(async (_, __, runSuper) => { | |||
await runSuper() | |||
}) | |||
|
|||
subtask("curve-swap", "Swap tokens using Curve 3Pool") |
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.
Nice!
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.
Thanks. I used that yesterday to swap DAI for USDC.
assetSymbol: ThreeCRV.symbol, | ||
} | ||
|
||
export const mvUSDC3PCV: Token = { |
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.
@toniocodo you might want this :p
* Vault deployment (#58) * chore: added config for new vaults on mainnet * chore: more Hardhat task refactoring * chore: Hardhat tasks to snap basic vault and convex meta vault details * chore: bump package version