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 backend provided slippage #5649
Conversation
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@rainbow-me/swaps@0.16.0 |
/** | ||
* NOTE: Before executing a swap, we want to check several things: | ||
* 1. assetToSell && assetToBuy addresses match the quote values | ||
* 2. outputNative matches quote value |
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.
this check makes sense but not sure if practical, like isnt there a case where the quote could update
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.
outputNative is updated when the quote updates
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.
Probably not a necessary check, but is more of a sanity check that the user is displayed and can confirm what will actually get received.
…eplink-add * 'develop' of github.com:rainbow-me/rainbow: simplify logic for fee display (#5660) zustand: use smol subscriptions (#5661) useSyncSharedValue (#5662) handle wc links in browser (#5652) Add default error page (#5657) update firebase (#5636) browser: fix android card images (#5651) Add backend provided slippage (#5649) Add rainbow fee to review panel (#5650) review prompt changes (#5656)
Fixes APP-1357
What changed (plus any additional context for devs)
Screen recordings / screenshots
What to test