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

Add Optimism, Base, Metis and Avalanche Info in Common Package #5

Merged

Conversation

Jeff-CCH
Copy link
Contributor

@Jeff-CCH Jeff-CCH commented Oct 6, 2023

  • changeset
  • network info
  • router info

@Jeff-CCH Jeff-CCH changed the title feat: add Optimism, Base, Metis and Avalanche info Support Optimism, Base, Metis and Avalanche Oct 6, 2023
@Jeff-CCH Jeff-CCH changed the title Support Optimism, Base, Metis and Avalanche Add Optimism, Base, Metis and Avalanche Info in Common Package Oct 6, 2023
@Jeff-CCH Jeff-CCH force-pushed the feat/support_Base_Metis_Optimism_Avalanche branch from ecd41c8 to 863e1b6 Compare October 6, 2023 07:06
@Jeff-CCH Jeff-CCH force-pushed the feat/support_Base_Metis_Optimism_Avalanche branch from 863e1b6 to 12ea0d9 Compare October 6, 2023 07:15
"symbol": "WETH",
"name": "Wrapped Ether"
},
"multicall2Address": "0x091e99cb1C49331a94dD62755D168E941AbD0693",
Copy link
Contributor

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.

Copy link
Contributor Author

@Jeff-CCH Jeff-CCH Oct 11, 2023

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.

Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

Copy link
Contributor

@chouandy chouandy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Jeff-CCH Jeff-CCH merged commit 141462f into dinngo:master Oct 11, 2023
2 of 3 checks passed
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