-
Notifications
You must be signed in to change notification settings - Fork 12
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 Optimism, Base, Metis and Avalanche Info in Common Package #5
Add Optimism, Base, Metis and Avalanche Info in Common Package #5
Conversation
Jeff-CCH
commented
Oct 6, 2023
•
edited
Loading
edited
- changeset
- network info
- router info
ecd41c8
to
863e1b6
Compare
863e1b6
to
12ea0d9
Compare
"symbol": "WETH", | ||
"name": "Wrapped Ether" | ||
}, | ||
"multicall2Address": "0x091e99cb1C49331a94dD62755D168E941AbD0693", |
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.
After checking, the contract interface at that address is not Multicall2. It is an optimized version modified by Uniswap. Since Multicall3 is backward compatible with Multicall2, we could use Multicall3 as a substitute for Multicall2 at the address.
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.
That's pretty weird since the Uniswap team put this address in the Multicall2 address column. Will use the multicall3Address
for the multicall2Address
.
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.
It seems like Uniswap has already transitioned to using its own Multicall. I think I should find time to refine how we retrieve quotes from their interface.
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.
fixed
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