-
Notifications
You must be signed in to change notification settings - Fork 180
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
fix: hardcode gas on celo #760
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Graphite Automations"Request reviewers once CI passes on routing-api repo" took an action on this PR • (06/25/24)6 reviewers were added and 1 assignee was added to this PR based on 's automation. |
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.
Can we pelase add a test here? This is similar to zksync right?
yes, and also requested by mobile team |
@cgkol added unit test |
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.
TY friend
the gas used proprietary SOR algorithm is too low on celo, but tenderly cannot simulate on celo yet to support accurate gas estimate. mobile launch on celo requires a gas used, that will not result in mass onchain swap failure. We determined that routing-api returns a hardcoded 5 million gas used on celo would avoid large onchain swap failures on mobile.
tested locally with uniswap-web source, gas estimate doesn't change: https://app.warp.dev/block/iYh1m8qV3oW917Xa0W9knf
tested locally with uniswap-ios source, gas estimate changed to 5000000: https://app.warp.dev/block/Tjl6BiNxcK1vkr1yDfhARc