-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
add missing factory to curve pools #6218
Conversation
Workflow run id 9598904390 approved. |
Workflow run id 9598904837 approved. |
Workflow run id 9598904840 approved. |
Hey, @jeff-dude! can you please help out with this failed test? |
whew, thank you soo much for this @viniabussafi!! |
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.
LGTM.✅
Thank you @viniabussafi
Workflow run id 9683614625 approved. |
Workflow run id 9683614636 approved. |
Workflow run id 9683614638 approved. |
Workflow run id 9683693733 approved. |
Workflow run id 9683693942 approved. |
Workflow run id 9683693940 approved. |
Workflow run id 9683716039 approved. |
Workflow run id 9683716296 approved. |
Workflow run id 9683716280 approved. |
Workflow run id 9683829603 approved. |
Workflow run id 9683829448 approved. |
Workflow run id 9683829605 approved. |
Thank you for contributing to Spellbook 🪄
Contribution type
Please check the type of contribution this pull request is for:
For adding to existing spell lineage
If you are adding to an existing spell lineage, please provide the following information:
This PR adds the CurveStableswapFactoryNG to the curvefi_ethereum_view_pools spell, since there are some missing pools on Curve's dex trades, such as https://etherscan.io/address/0xdb74dfdd3bb46be8ce6c33dc9d82777bcfc3ded5
@mendesfabio