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.

Sign up for GitHub

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: Update the document deploys to current #683

Merged
merged 4 commits into from
Apr 10, 2024
Merged

Fix: Update the document deploys to current #683

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ec49f24
Fix: Update the document deploys to current
Rachel-Eichenberger Mar 26, 2024
da5fbb7
Fix: Clean up commit
Rachel-Eichenberger Mar 27, 2024
13a81c4
Fix: Remove Goerli
Rachel-Eichenberger Mar 28, 2024
3ebd8c5
fix: Add Blast Network
Rachel-Eichenberger Apr 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 12 additions & 10 deletions docs/contracts/v2/reference/smart-contracts/08-deployment-addresses.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,16 @@ title: V2 Deployment Addresses

# Uniswap v2 Contract Deployments

Contract addresses for the [Uniswap V2 Factory](https://github.com/Uniswap/v2-core/blob/master/contracts/UniswapV2Factory.sol)
Contract addresses for the [`Uniswap V2 Factory`](https://github.com/Uniswap/v2-core/blob/master/contracts/UniswapV2Factory.sol) and the [`Uniswap V2Router02`](https://github.com/Uniswap/v2-periphery/blob/master/contracts/UniswapV2Router02.sol)

| Network | Contract Address |
| ---------------------------------------------------- | ----------------------------------------------- |
| Mainnet, Ropsten, Rinkeby, Goerli, Kovan | `0x5C69bEe701ef814a2B6a3EDD4B1652CB9cc5aA6f` |
| Arbitrum | `0xf1D7CC64Fb4452F05c498126312eBE29f30Fbcf9` |
| Avalanche | `0x9e5A52f57b3038F1B8EeE45F28b3C1967e22799C` |
| BNB Chain | `0x8909Dc15e40173Ff4699343b6eB8132c65e18eC6` |
| Base | `0x8909Dc15e40173Ff4699343b6eB8132c65e18eC6` |
| Optimism | `0x0c3c1c532F1e39EdF36BE9Fe0bE1410313E074Bf` |
| Polygon | `0x9e5A52f57b3038F1B8EeE45F28b3C1967e22799C` |
| Network | Factory Contract Address | V2Router02 Contract Address |
| ---------------------------------------------------- | -------------------------------------------- | -------------------------------------------- |
| Mainnet | `0x5C69bEe701ef814a2B6a3EDD4B1652CB9cc5aA6f` | `0x7a250d5630B4cF539739dF2C5dAcb4c659F2488D` |
| Etherum Sepolia | `0xB7f907f7A9eBC822a80BD25E224be42Ce0A698A0` | `0x425141165d3DE9FEC831896C016617a52363b687` |
Copy link
Contributor

Choose a reason for hiding this comment

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

the v2router02 contract here for Sepolia is not verified on etherscan, should we try to get that verified before publishing this?

Copy link
Contributor

Choose a reason for hiding this comment

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

i'm also not seeing any transactions there, so just double checking that it's correct

Copy link
Member

Choose a reason for hiding this comment

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

Prior deployed UniswapV2Router02 0x7a250d5630B4cF539739dF2C5dAcb4c659F2488 on Sepolia was verified, but it doesn't have the v2 liquidity support. 0x425141165d3DE9FEC831896C016617a52363b687 was retroactively deployed to Sepolia to support v2 liquidity. Probably the deploy was done in a rush so it was not verified.

I believe this was part of the v2-deploy-everywhere, where UniswapV2Router02 needs to support v2 liquidity as well.

Copy link
Member

Choose a reason for hiding this comment

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

And I think the reason there's no transaction yet is because the PR hasn't merged yet.Uniswap/sdk-core#131

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This was newly launched by Emily https://uniswapteam.slack.com/archives/C05MF1J6GR5/p1709845257901909?thread_ts=1708991083.674689&cid=C05MF1J6GR5
The script errored trying to verify the code. Yes, Blast put us behind on pushing the PR.

| Arbitrum | `0xf1D7CC64Fb4452F05c498126312eBE29f30Fbcf9` | `0x4752ba5dbc23f44d87826276bf6fd6b1c372ad24` |
| Avalanche | `0x9e5A52f57b3038F1B8EeE45F28b3C1967e22799C` | `0x4752ba5dbc23f44d87826276bf6fd6b1c372ad24` |
| BNB Chain | `0x8909Dc15e40173Ff4699343b6eB8132c65e18eC6` | `0x4752ba5dbc23f44d87826276bf6fd6b1c372ad24` |
| Base | `0x8909Dc15e40173Ff4699343b6eB8132c65e18eC6` | `0x4752ba5dbc23f44d87826276bf6fd6b1c372ad24` |
| Optimism | `0x0c3c1c532F1e39EdF36BE9Fe0bE1410313E074Bf` | `0x4A7b5Da61326A6379179b40d00F57E5bbDC962c2` |
| Polygon | `0x9e5A52f57b3038F1B8EeE45F28b3C1967e22799C` | `0xedf6066a2b290C185783862C7F4776A2C8077AD1` |
just-toby marked this conversation as resolved.
Show resolved Hide resolved
| Blast | `0x5C346464d33F90bABaf70dB6388507CC889C1070` | `0xBB66Eb1c5e875933D44DAe661dbD80e5D9B03035` |
86 changes: 0 additions & 86 deletions docs/contracts/v3/reference/Deployments.md
View file
Open in desktop

This file was deleted.

Loading
Loading