Skip to content

Conversation

@cbachmeier
Copy link
Contributor

@cbachmeier cbachmeier commented May 19, 2023

Description

  • Adds Sepolia tesnet w/ chain id: 11155111, important resources, and relevant contract addresses
  • Updates token-list, SOR, and UR
  • Search, Token Discover page, and Token Details pages all show mainnet values, same as we do for goerli

Slack thread:

Relevant docs:

Screen capture

Successful Pool
Screen Shot 2023-05-31 at 16 33 47

Successful swap
Screen Shot 2023-05-31 at 16 31 01

Test plan

QA (ie manual testing)

@vercel
Copy link

vercel bot commented May 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
interface ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 1, 2023 9:55pm
interface-node18 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 1, 2023 9:55pm

@cbachmeier cbachmeier temporarily deployed to false May 19, 2023 22:52 — with GitHub Actions Inactive
@cbachmeier cbachmeier temporarily deployed to false May 19, 2023 22:52 — with GitHub Actions Inactive
@cbachmeier cbachmeier temporarily deployed to false May 19, 2023 22:56 — with GitHub Actions Inactive
@cbachmeier cbachmeier temporarily deployed to false May 19, 2023 22:56 — with GitHub Actions Inactive
@cbachmeier cbachmeier temporarily deployed to false May 19, 2023 22:56 — with GitHub Actions Inactive
@cbachmeier cbachmeier temporarily deployed to false May 19, 2023 22:56 — with GitHub Actions Inactive
@cypress
Copy link

cypress bot commented May 19, 2023

Passing run #11581 ↗︎

0 69 8 0 Flakiness 0

Details:

feat: Add support for Sepolia
Project: Uniswap Interface Commit: de7b2252dc
Status: Passed Duration: 03:39 💡
Started: Jun 1, 2023 9:59 PM Ended: Jun 1, 2023 10:02 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@codecov
Copy link

codecov bot commented May 19, 2023

Codecov Report

Patch coverage: 90.90% and project coverage change: -0.92 ⚠️

Comparison is base (8954aa7) 60.40% compared to head (715f213) 59.48%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6610      +/-   ##
==========================================
- Coverage   60.40%   59.48%   -0.92%     
==========================================
  Files         732      732              
  Lines       21463    21473      +10     
  Branches     7091     7092       +1     
==========================================
- Hits        12964    12773     -191     
- Misses       8425     8624     +199     
- Partials       74       76       +2     
Flag Coverage Δ
e2e-tests 59.47% <90.90%> (-1.09%) ⬇️
unit-tests 22.74% <66.66%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/constants/chainInfo.ts 88.88% <ø> (ø)
src/constants/chains.ts 100.00% <ø> (ø)
src/constants/networks.ts 75.00% <ø> (ø)
src/constants/providers.ts 100.00% <ø> (ø)
src/constants/routing.ts 100.00% <ø> (ø)
src/graphql/data/util.tsx 18.51% <ø> (ø)
src/hooks/useERC20Permit.ts 33.33% <ø> (ø)
src/utils/anonymizeLink.ts 90.00% <ø> (ø)
src/utils/getExplorerLink.ts 50.00% <ø> (ø)
src/utils/switchChain.ts 5.00% <0.00%> (-0.27%) ⬇️
... and 3 more

... and 26 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@socket-security
Copy link

socket-security bot commented May 23, 2023

New dependency changes detected. Learn more about Socket for GitHub ↗︎


👍 No new dependency issues detected in pull request

Bot Commands

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore foo@1.0.0 bar@* or ignore all packages with @SocketSecurity ignore-all

Pull request alert summary
Issue Status
Install scripts ✅ 0 issues
Native code ✅ 0 issues
Bin script shell injection ✅ 0 issues
Unresolved require ✅ 0 issues
Invalid package.json ✅ 0 issues
HTTP dependency ✅ 0 issues
Git dependency ✅ 0 issues
Potential typo squat ✅ 0 issues
Known Malware ✅ 0 issues
Telemetry ✅ 0 issues
Protestware/Troll package ✅ 0 issues

📊 Modified Dependency Overview:

⬆️ Updated Package Version Diff Added Capability Access +/- Transitive Count Publisher
@uniswap/universal-router-sdk@1.5.1 1.3.9...1.5.1 None +1/-4 alicehenshaw
@uniswap/default-token-list@9.6.0 9.4.0...9.6.0 None +0/-0 matteen
@uniswap/smart-order-router@3.12.1 3.6.1...3.12.1 environment +2/-6 mikeki

🚮 Removed packages: @uniswap/sdk-core@3.2.2

@JFrankfurt
Copy link
Contributor

After adding liquidity to the ETH:UNI 0.3% pool I got this message when my transaction confirmed. I don't thiiiink it's related? Prior to the transaction the .3 pool indicated it had already been created.
image

@cbachmeier
Copy link
Contributor Author

I'm going through #6036 to evaluate parity, it looks like you may want to add: sepolia.etherscan.io to src/utils/anonymizeLink.ts and src/utils/getExplorerLink.ts

Sepolia is already added to BLOCK_EXPLORER_PREFIXES in getExplorerLink and thanks for the callout for anonymizeLink, I've Sepolia to it

@cbachmeier
Copy link
Contributor Author

After adding liquidity to the ETH:UNI 0.3% pool I got this message when my transaction confirmed. I don't thiiiink it's related? Prior to the transaction the .3 pool indicated it had already been created. image

@JFrankfurt Are you able to replicate? I can't seem to get the same behaviour for the same pool.

Copy link
Contributor

@cmcewen cmcewen left a comment

Choose a reason for hiding this comment

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

wooooot!

const ARBITRUM_GOERLI_TICK_LENS_ADDRESSES = '0xb52429333da969a0C79a60930a4Bf0020E5D1DE8'

// sepolia v3 addresses
const SEPOLIA_V3_CORE_FACTORY_ADDRESSES = '0x0227628f3F023bb0B980b67D528571c95c6DaC1c'
Copy link
Contributor

Choose a reason for hiding this comment

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

not relevant to PR - but kinda surprised these don't live in the SDK

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This does feel like a process that could be part of the protocol deployment scripts to save a lot of manual entry.

[SupportedChainId.GOERLI]: new Token(SupportedChainId.GOERLI, UNI_ADDRESS[5], 18, 'UNI', 'Uniswap'),
[SupportedChainId.MAINNET]: new Token(
SupportedChainId.MAINNET,
UNI_ADDRESS[SupportedChainId.MAINNET],
Copy link
Contributor

Choose a reason for hiding this comment

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

nice 👍

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.

5 participants