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.

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

Update pool-exits.md #170

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

OdysseusTheKing
Copy link

Fixed Encoding section for Proportional and Custom Exits.

Determined and confirmed through testing that these two definitions were mixed up. See the below transactions on Metis Goerli and note the userData fields for further proof. Contract addresses also included for reference.

Proportional Exit (returning all 3 specified tokens back from estimated BPT amount): https://goerli.explorer.metisdevops.link/tx/0x9b9e873461007d2fb0950d24a22eff87cfb4647c1f5d89c19867d1551bec1463

Proportional Exit (returning only 2 specified tokens back from estimated BPT amount): https://goerli.explorer.metisdevops.link/tx/0x2ea019ecc160448689100f3d46bbccba7b9eece05d579447d390e0978e157a24

Custom Exit (returning estimated amounts of all 3 tokens from exact BPT amount): https://goerli.explorer.metisdevops.link/tx/0x21af9d0f17773d134e9384098864844c11baf35782577af14058357fce10b1be

Vault: https://goerli.explorer.metisdevops.link/address/0xAd8Ab48eD57F35dCD2a0c67Bfc5c91d35bF1C5b4
Tri Stable ComposableStablePoolV5 (containing mock ERCs for USDC/USDT/DAI): https://goerli.explorer.metisdevops.link/address/0xBf49cC8510040B71AF73bD4A9fcEB70EBDA0C108

Fixed Encoding section for Proportional and Custom Exits.
Copy link

vercel bot commented Nov 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 16, 2023 9:34am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants