fix: block ip infringing collection (#7614)

hotfix: fix: android banner DownloadButton onClick should not propagate up #7611

CODEOWNERS

@@ -0,0 +1 @@
+* @uniswap/web-admins

cypress/e2e/swap/errors.test.ts

@@ -1,6 +1,7 @@
 import { BigNumber } from '@ethersproject/bignumber'
 import { InterfaceSectionName } from '@uniswap/analytics-events'
 import { CurrencyAmount } from '@uniswap/sdk-core'
+import { FeatureFlag } from 'featureFlags'
 
 import { DEFAULT_DEADLINE_FROM_NOW } from '../../../src/constants/misc'
 import { DAI, USDC_MAINNET } from '../../../src/constants/tokens'
@@ -64,7 +65,9 @@ describe('Swap errors', () => {
   })
 
   it('slippage failure', () => {
-    cy.visit(`/swap?inputCurrency=${USDC_MAINNET.address}&outputCurrency=${DAI.address}`)
+    cy.visit(`/swap?inputCurrency=${USDC_MAINNET.address}&outputCurrency=${DAI.address}`, {
+      featureFlags: [{ name: FeatureFlag.uniswapXDefaultEnabled, value: false }],
+    })
     cy.hardhat({ automine: false }).then(async (hardhat) => {
       await hardhat.fund(hardhat.wallet, CurrencyAmount.fromRawAmount(USDC_MAINNET, 500e6))
       await hardhat.mine()
@@ -87,6 +90,7 @@ describe('Swap errors', () => {
       cy.get(getTestSelector('open-settings-dialog-button')).click()
       cy.get(getTestSelector('max-slippage-settings')).click()
       cy.get(getTestSelector('slippage-input')).clear().type('0.01')
+      cy.get(getTestSelector('toggle-uniswap-x-button')).click() // turn off uniswapx
       cy.get('body').click('topRight') // close modal
       cy.get(getTestSelector('slippage-input')).should('not.exist')
 

cypress/e2e/swap/fees.test.ts

@@ -4,7 +4,7 @@ import { FeatureFlag } from 'featureFlags'
 import { USDC_MAINNET } from '../../../src/constants/tokens'
 import { getBalance, getTestSelector } from '../../utils'
 
-describe('Swap with fees', () => {
+describe.skip('Swap with fees', () => {
   describe('Classic swaps', () => {
     beforeEach(() => {
       cy.visit('/swap', { featureFlags: [{ name: FeatureFlag.feesEnabled, value: true }] })

cypress/e2e/swap/uniswapx.test.ts

@@ -42,7 +42,8 @@ function stubSwapTxReceipt() {
   })
 }
 
-describe('UniswapX Toggle', () => {
+// TODO: FIX THESE TESTS where we should NOT stub for pricing requests
+describe.skip('UniswapX Toggle', () => {
   beforeEach(() => {
     stubNonPriceQuoteWith(QuoteWhereUniswapXIsBetter)
     cy.visit(`/swap/?inputCurrency=${USDC_MAINNET.address}&outputCurrency=${DAI.address}`, {
@@ -90,7 +91,7 @@ describe('UniswapX Toggle', () => {
   })
 })
 
-describe('UniswapX Orders', () => {
+describe.skip('UniswapX Orders', () => {
   beforeEach(() => {
     stubNonPriceQuoteWith(QuoteWhereUniswapXIsBetter)
     cy.intercept(OrderSubmissionEndpoint, { fixture: 'uniswapx/orderResponse.json' })
@@ -183,7 +184,7 @@ describe('UniswapX Orders', () => {
   })
 })
 
-describe('UniswapX Eth Input', () => {
+describe.skip('UniswapX Eth Input', () => {
   beforeEach(() => {
     stubNonPriceQuoteWith(QuoteWithEthInput)
     cy.intercept(OrderSubmissionEndpoint, { fixture: 'uniswapx/orderResponse.json' })
@@ -236,7 +237,7 @@ describe('UniswapX Eth Input', () => {
     cy.contains('Swapped')
   })
 
-  it('switches swap input to WETH after wrap', () => {
+  it('keeps ETH as the input currency before wrap completes', () => {
     // Setup a swap
     cy.get('#swap-currency-input .token-amount-input').type('1')
     cy.wait('@quote')
@@ -250,16 +251,25 @@ describe('UniswapX Eth Input', () => {
 
     // Close review modal before wrap is confirmed on chain
     cy.get(getTestSelector('confirmation-close-icon')).click()
+    // Confirm ETH is still the input token before wrap succeeds
+    cy.contains('ETH')
+  })
+
+  it('switches swap input to WETH after wrap', () => {
+    // Setup a swap
+    cy.get('#swap-currency-input .token-amount-input').type('1')
+    cy.wait('@quote')
+
+    // Prompt ETH wrap and confirm
+    cy.get('#swap-button').click()
+    cy.contains('Confirm swap').click()
+    cy.wait('@eth_sendRawTransaction')
     cy.hardhat().then((hardhat) => hardhat.mine())
 
     // Confirm wrap is successful and WETH is now input token
     cy.contains('Wrapped')
     cy.contains('WETH')
 
-    // Reopen review modal and continue swap
-    cy.get('#swap-button').click()
-    cy.contains('Confirm swap').click()
-
     // Approve WETH spend
     cy.wait('@eth_sendRawTransaction')
     cy.hardhat().then((hardhat) => hardhat.mine())
@@ -274,10 +284,15 @@ describe('UniswapX Eth Input', () => {
 
     // Verify swap success
     cy.contains('Swapped')
+
+    // Close modal
+    cy.get(getTestSelector('confirmation-close-icon')).click()
+    // The input currency should now be WETH
+    cy.contains('WETH')
   })
 })
 
-describe('UniswapX activity history', () => {
+describe.skip('UniswapX activity history', () => {
   beforeEach(() => {
     cy.intercept(QuoteEndpoint, { fixture: QuoteWhereUniswapXIsBetter })
     cy.intercept(OrderSubmissionEndpoint, { fixture: 'uniswapx/orderResponse.json' })

functions/api/image/nfts/collection/collectionImage.test.ts

@@ -7,19 +7,19 @@ const collectionImageUrls = [
   'http://127.0.0.1:3000/api/image/nfts/collection/0x49cf6f5d44e70224e2e23fdcdd2c053f30ada28b',
 ]
 
-const nonexistentImageUrls = [
-  'http://127.0.0.1:3000/api/image/nfts/collection/0xed5af388653567af2f388e6224dc7c4b3241c545',
-]
-
-test.each([...collectionImageUrls, ...nonexistentImageUrls])('collectionImageUrl', async (url) => {
+test.each([...collectionImageUrls])('collectionImageUrl', async (url) => {
   const response = await fetch(new Request(url))
   expect(response.status).toBe(200)
   expect(response.headers.get('content-type')).toBe('image/png')
 })
 
+const nonexistentImageUrls = [
+  'http://127.0.0.1:3000/api/image/nfts/collection/0xed5af388653567af2f388e6224dc7c4b3241c545',
+]
+
 const invalidCollectionImageUrls = ['http://127.0.0.1:3000/api/image/nfts/collection/0xd3adb33f']
 
-test.each(invalidCollectionImageUrls)('invalidAssetImageUrl', async (url) => {
+test.each([...invalidCollectionImageUrls, ...nonexistentImageUrls])('invalidAssetImageUrl', async (url) => {
   const response = await fetch(new Request(url))
   expect(response.status).toBeOneOf([404, 500])
 })

functions/nfts/collection/__snapshots__/collection.test.ts.snap

@@ -443,151 +443,3 @@ exports[`should inject metadata for collections 3`] = `
 </html>
 "
 `;
-
-exports[`should inject metadata for collections 4`] = `
-"<!DOCTYPE html>
-<html translate="no">
-  <head>
-    <meta charset="utf-8" />
-
-    <title>Uniswap Interface</title>
-
-    <!--
-       will be replaced with the URL of the \`public\` folder during build.
-      Only files inside the \`public\` folder can be referenced from the HTML.
-    -->
-    <link rel="shortcut icon" type="image/png" href="/favicon.png" />
-    <link rel="apple-touch-icon" sizes="192x192" href="/images/192x192_App_Icon.png" />
-    <link rel="apple-touch-icon" sizes="512x512" href="/images/512x512_App_Icon.png" />
-
-    <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no" />
-    <meta name="theme-color" content="#fff" />
-    <meta
-      http-equiv="Content-Security-Policy"
-      
-        content="script-src 'self' 'unsafe-inline'"
-      
-    />
-
-    <!-- 
-      Apple Smart App Banner for Safari on iOS
-      https://developer.apple.com/documentation/webkit/promoting_apps_with_smart_app_banners
-     -->
-    <meta name="apple-itunes-app" content="app-id=6443944476">
-
-    <!--
-      manifest.json provides metadata used when the app is installed as a PWA.
-      See https://developers.google.com/web/fundamentals/web-app-manifest/
-    -->
-    <link rel="manifest" href="/manifest.json" />
-
-    <link rel="preconnect" href="https://api.uniswap.org/" crossorigin/>
-    <link rel="preconnect" href="https://mainnet.infura.io/" crossorigin/>
-
-    <link rel="preload" href="/fonts/Basel-Book.woff" as="font" type="font/woff" crossorigin />
-    <link rel="preload" href="/fonts/Basel-Book.woff2" as="font" type="font/woff2" crossorigin />
-    <link rel="preload" href="/fonts/Basel-Medium.woff" as="font" type="font/woff" crossorigin />
-    <link rel="preload" href="/fonts/Basel-Medium.woff2" as="font" type="font/woff2" crossorigin />
-   
-    <style>
-      * {
-        font-family: 'Basel', sans-serif;
-        box-sizing: border-box;
-      }
-
-      /**
-          Explicitly load Basel var from public/ so it does not block LCP's critical path.
-        */
-      @font-face {
-        font-family: 'Basel';
-        font-weight: 535;
-        font-style: normal;
-        font-display: block;
-        src: 
-          url('/fonts/Basel-Medium.woff2') format('woff2'),
-          url('/fonts/Basel-Medium.woff') format('woff');
-      }
-
-      @font-face {
-        font-family: 'Basel';
-        font-weight: 485;
-        font-style: normal;
-        font-display: block;
-        src: 
-          url('/fonts/Basel-Book.woff') format('woff2'),
-          url('/fonts/Basel-Book.woff') format('woff');
-      }
-
-      @supports (font-variation-settings: normal) {
-        * {
-          font-family: 'Basel', sans-serif;
-        }
-      }
-
-      html,
-      body {
-        margin: 0;
-        padding: 0;
-      }
-
-      button {
-        user-select: none;
-      }
-
-      html {
-        font-size: 16px;
-        font-weight: 485;
-        font-variant: none;
-        font-smooth: always;
-        text-rendering: optimizeLegibility !important;
-        -webkit-font-smoothing: antialiased !important;
-        -moz-osx-font-smoothing: grayscale;
-        -webkit-tap-highlight-color: rgba(0, 0, 0, 0);
-      }
-
-      /* Use this to apply network-specific gradient backgrounds, in RadialGradientByChainUpdater.ts */
-      #background-radial-gradient {
-        position: fixed;
-        top: 0;
-        left: 0;
-        right: 0;
-        pointer-events: none;
-        width: 200vw;
-        height: 200vh;
-        transform: translate(-50vw, -100vh);
-        z-index: -1;
-      }
-
-      html,
-      body,
-      #root {
-        min-height: 100%;
-      }
-
-      @media (prefers-color-scheme: dark) {
-        html {
-          background: linear-gradient(rgb(19, 19, 19) 0%, rgb(19, 19, 19) 100%);
-        }
-      }
-
-      @media (prefers-color-scheme: light) {
-        html {
-          background: radial-gradient(100% 100% at 50% 0%, rgba(255, 184, 226, 0) 0%, rgba(255, 255, 255, 0) 100%), rgb(255, 255, 255);
-        }
-      }
-    </style>
-  <script defer src="/static/js/bundle.js"></script><meta property="og:title" content="0xed5af388653567af2f388e6224dc7c4b3241c545 on Uniswap"/><meta property="og:image" content="http://127.0.0.1:3000/api/image/nfts/collection/0xed5af388653567af2f388e6224dc7c4b3241c545"/><meta property="og:image:width" content="1200"/><meta property="og:image:height" content="630"/><meta property="og:image:alt" content="0xed5af388653567af2f388e6224dc7c4b3241c545 on Uniswap"/><meta property="og:type" content="website"/><meta property="og:url" content="http://127.0.0.1:3000/nfts/collection/0xed5af388653567af2f388e6224dc7c4b3241c545"/><meta property="twitter:card" content="summary_large_image"/><meta property="twitter:title" content="0xed5af388653567af2f388e6224dc7c4b3241c545 on Uniswap"/><meta property="twitter:image" content="http://127.0.0.1:3000/api/image/nfts/collection/0xed5af388653567af2f388e6224dc7c4b3241c545"/><meta property="twitter:image:alt" content="0xed5af388653567af2f388e6224dc7c4b3241c545 on Uniswap"/></head>
-
-  <body>
-    <noscript>You need to enable JavaScript to run this app.</noscript>
-
-    <div id="root">
-      <!-- Triggers the font to load immediately and then is replaced by the app -->
-      <div>&emsp;</div>
-    </div>
-
-    <div id="background-radial-gradient"></div>
-  </body>
-</html>
-"
-`;

functions/nfts/collection/collection.test.ts

@@ -16,15 +16,7 @@ const collections = [
   },
 ]
 
-const nonexistentCollections = [
-  {
-    address: '0xed5af388653567af2f388e6224dc7c4b3241c545',
-    collectionName: '0xed5af388653567af2f388e6224dc7c4b3241c545',
-    image: 'http://127.0.0.1:3000/api/image/nfts/collection/0xed5af388653567af2f388e6224dc7c4b3241c545',
-  },
-]
-
-test.each([...collections, ...nonexistentCollections])('should inject metadata for collections', async (collection) => {
+test.each([...collections])('should inject metadata for collections', async (collection) => {
   const url = 'http://127.0.0.1:3000/nfts/collection/' + collection.address
   const body = await fetch(new Request(url)).then((res) => res.text())
   expect(body).toMatchSnapshot()
@@ -42,24 +34,33 @@ test.each([...collections, ...nonexistentCollections])('should inject metadata f
   expect(body).toContain(`<meta property="twitter:image:alt" content="${collection.collectionName} on Uniswap"/>`)
 })
 
+const nonexistentCollections = [
+  {
+    address: '0xed5af388653567af2f388e6224dc7c4b3241c545',
+  },
+]
+
 const invalidCollections = [
   {
     address: '0xd3adb33f',
   },
 ]
 
-test.each(invalidCollections)('should not inject metadata for nonexistent collections', async (collection) => {
-  const url = 'http://127.0.0.1:3000/nfts/collection/' + collection.address
-  const body = await fetch(new Request(url)).then((res) => res.text())
-  expect(body).not.toContain('og:title')
-  expect(body).not.toContain('og:image')
-  expect(body).not.toContain('og:image:width')
-  expect(body).not.toContain('og:image:height')
-  expect(body).not.toContain('og:type')
-  expect(body).not.toContain('og:url')
-  expect(body).not.toContain('og:image:alt')
-  expect(body).not.toContain('twitter:card')
-  expect(body).not.toContain('twitter:title')
-  expect(body).not.toContain('twitter:image')
-  expect(body).not.toContain('twitter:image:alt')
-})
+test.each([...invalidCollections, ...nonexistentCollections])(
+  'should not inject metadata for nonexistent collections',
+  async (collection) => {
+    const url = 'http://127.0.0.1:3000/nfts/collection/' + collection.address
+    const body = await fetch(new Request(url)).then((res) => res.text())
+    expect(body).not.toContain('og:title')
+    expect(body).not.toContain('og:image')
+    expect(body).not.toContain('og:image:width')
+    expect(body).not.toContain('og:image:height')
+    expect(body).not.toContain('og:type')
+    expect(body).not.toContain('og:url')
+    expect(body).not.toContain('og:image:alt')
+    expect(body).not.toContain('twitter:card')
+    expect(body).not.toContain('twitter:title')
+    expect(body).not.toContain('twitter:image')
+    expect(body).not.toContain('twitter:image:alt')
+  }
+)

src/components/Banner/AndroidAnnouncementBanner/index.tsx

@@ -51,7 +51,12 @@ export default function AndroidAnnouncementBanner() {
           <ThemedText.LabelMicro>
             <Trans>Available now - download from the Google Play Store today</Trans>
           </ThemedText.LabelMicro>
-          <DownloadButton onClick={onClick}>
+          <DownloadButton
+            onClick={(e) => {
+              e.stopPropagation()
+              onClick()
+            }}
+          >
             <Trans>Download now</Trans>
           </DownloadButton>
         </TextContainer>

src/components/swap/ConfirmSwapModal.tsx

@@ -31,7 +31,6 @@ import { ThemedText } from 'theme/components'
 import invariant from 'tiny-invariant'
 import { isL2ChainId } from 'utils/chains'
 import { SignatureExpiredError } from 'utils/errors'
-import { NumberType, useFormatter } from 'utils/formatNumbers'
 import { formatSwapPriceUpdatedEventProperties } from 'utils/loggingFormatters'
 import { didUserReject } from 'utils/swapErrorToUserReadableMessage'
 import { tradeMeaningfullyDiffers } from 'utils/tradeMeaningFullyDiffer'
@@ -82,7 +81,6 @@ function useConfirmModalState({
   const [confirmModalState, setConfirmModalState] = useState<ConfirmModalState>(ConfirmModalState.REVIEWING)
   const [approvalError, setApprovalError] = useState<PendingModalError>()
   const [pendingModalSteps, setPendingModalSteps] = useState<PendingConfirmModalState[]>([])
-  const { formatCurrencyAmount } = useFormatter()
 
   // This is a function instead of a memoized value because we do _not_ want it to update as the allowance changes.
   // For example, if the user needs to complete 3 steps initially, we should always show 3 step indicators
@@ -117,14 +115,7 @@ function useConfirmModalState({
   const nativeCurrency = useNativeCurrency(chainId)
 
   const [wrapTxHash, setWrapTxHash] = useState<string>()
-  const { execute: onWrap } = useWrapCallback(
-    nativeCurrency,
-    trade.inputAmount.currency,
-    formatCurrencyAmount({
-      amount: trade.inputAmount,
-      type: NumberType.SwapTradeAmount,
-    })
-  )
+  const { execute: onWrap } = useWrapCallback(nativeCurrency, trade.inputAmount.currency, trade.inputAmount.toExact())
   const wrapConfirmed = useIsTransactionConfirmed(wrapTxHash)
   const prevWrapConfirmed = usePrevious(wrapConfirmed)
   const catchUserReject = async (e: any, errorType: PendingModalError) => {
@@ -142,9 +133,6 @@ function useConfirmModalState({
           onWrap?.()
             .then((wrapTxHash) => {
               setWrapTxHash(wrapTxHash)
-              // After the wrap has succeeded, reset the input currency to be WETH
-              // because the trade will be on WETH -> token
-              onCurrencySelection(Field.INPUT, trade.inputAmount.currency)
               sendAnalyticsEvent(InterfaceEventName.WRAP_TOKEN_TXN_SUBMITTED, {
                 chain_id: chainId,
                 token_symbol: maximumAmountIn?.currency.symbol,
@@ -192,7 +180,6 @@ function useConfirmModalState({
       onWrap,
       trace,
       trade,
-      onCurrencySelection,
     ]
   )
 
@@ -209,10 +196,20 @@ function useConfirmModalState({
   useEffect(() => {
     // If the wrapping step finished, trigger the next step (allowance or swap).
     if (wrapConfirmed && !prevWrapConfirmed) {
+      // After the wrap has succeeded, reset the input currency to be WETH
+      // because the trade will be on WETH -> token
+      onCurrencySelection(Field.INPUT, trade.inputAmount.currency)
       // moves on to either approve WETH or to swap submission
       performStep(pendingModalSteps[1])
     }
-  }, [pendingModalSteps, performStep, prevWrapConfirmed, wrapConfirmed])
+  }, [
+    pendingModalSteps,
+    performStep,
+    prevWrapConfirmed,
+    wrapConfirmed,
+    onCurrencySelection,
+    trade.inputAmount.currency,
+  ])
 
   useEffect(() => {
     if (

src/connection/WalletConnectV2.ts

@@ -3,7 +3,7 @@ import { URI_AVAILABLE, WalletConnect, WalletConnectConstructorArgs } from '@web
 import { sendAnalyticsEvent } from 'analytics'
 import { L1_CHAIN_IDS, L2_CHAIN_IDS } from 'constants/chains'
 import { Z_INDEX } from 'theme/zIndex'
-import { isIOS } from 'utils/userAgent'
+import { isAndroid, isIOS } from 'utils/userAgent'
 
 import { RPC_URLS } from '../constants/networks'
 
@@ -83,7 +83,7 @@ export class UniwalletConnect extends WalletConnectV2 {
       this.events.emit(UniwalletConnect.UNI_URI_AVAILABLE, `https://uniswap.org/app/wc?uri=${uri}`)
 
       // Opens deeplink to Uniswap Wallet if on iOS
-      if (isIOS) {
+      if (isIOS || isAndroid) {
         // Using window.location.href to open the deep link ensures smooth navigation and leverages OS handling for installed apps,
         // avoiding potential popup blockers or inconsistent behavior associated with window.open
         window.location.href = `uniswap://wc?uri=${encodeURIComponent(uri)}`

src/constants/tokens.ts

@@ -90,7 +90,13 @@ export const MATIC_MAINNET = new Token(
   'MATIC',
   'Polygon Matic'
 )
-const MATIC_POLYGON = new Token(ChainId.POLYGON, '0x0000000000000000000000000000000000001010', 18, 'MATIC', 'Matic')
+export const MATIC_POLYGON = new Token(
+  ChainId.POLYGON,
+  '0x0000000000000000000000000000000000001010',
+  18,
+  'MATIC',
+  'Matic'
+)
 export const DAI_POLYGON = new Token(
   ChainId.POLYGON,
   '0x8f3Cf7ad23Cd3CaDbD9735AFf958023239c6A063',
@@ -143,13 +149,6 @@ export const WBTC_OPTIMISM = new Token(
   'WBTC',
   'Wrapped BTC'
 )
-const MATIC_POLYGON_MUMBAI = new Token(
-  ChainId.POLYGON_MUMBAI,
-  '0x0000000000000000000000000000000000001010',
-  18,
-  'MATIC',
-  'Matic'
-)
 export const WETH_POLYGON_MUMBAI = new Token(
   ChainId.POLYGON_MUMBAI,
   '0xa6fa4fb5f76172d178d61b04b0ecd319c5d1c0aa',
@@ -359,14 +358,21 @@ export function isPolygon(chainId: number): chainId is ChainId.POLYGON | ChainId
   return chainId === ChainId.POLYGON_MUMBAI || chainId === ChainId.POLYGON
 }
 
-function getPolygonNativeCurrency(chainId: number) {
-  switch (chainId) {
-    case ChainId.POLYGON:
-      return MATIC_POLYGON
-    case ChainId.POLYGON_MUMBAI:
-      return MATIC_POLYGON_MUMBAI
-    default:
-      throw new Error('Not polygon')
+class PolygonNativeCurrency extends NativeCurrency {
+  equals(other: Currency): boolean {
+    return other.isNative && other.chainId === this.chainId
+  }
+
+  get wrapped(): Token {
+    if (!isPolygon(this.chainId)) throw new Error('Not Polygon')
+    const wrapped = WRAPPED_NATIVE_CURRENCY[this.chainId]
+    invariant(wrapped instanceof Token)
+    return wrapped
+  }
+
+  public constructor(chainId: number) {
+    if (!isPolygon(chainId)) throw new Error('Not Polygon')
+    super(chainId, 18, 'MATIC', 'Matic')
   }
 }
 
@@ -433,7 +439,7 @@ export function nativeOnChain(chainId: number): NativeCurrency | Token {
   if (cachedNativeCurrency[chainId]) return cachedNativeCurrency[chainId]
   let nativeCurrency: NativeCurrency | Token
   if (isPolygon(chainId)) {
-    nativeCurrency = getPolygonNativeCurrency(chainId)
+    nativeCurrency = new PolygonNativeCurrency(chainId)
   } else if (isCelo(chainId)) {
     nativeCurrency = getCeloNativeCurrency(chainId)
   } else if (isBsc(chainId)) {

src/hooks/useAccountRiskCheck.ts

@@ -1,4 +1,3 @@
-import ms from 'ms'
 import { useEffect } from 'react'
 import { ApplicationModal, setOpenModal } from 'state/application/reducer'
 import { useAppDispatch } from 'state/hooks'
@@ -7,34 +6,23 @@ export default function useAccountRiskCheck(account: string | null | undefined)
   const dispatch = useAppDispatch()
 
   useEffect(() => {
-    if (account) {
-      const riskCheckLocalStorageKey = `risk-check-${account}`
-      const now = Date.now()
-      try {
-        // Check local browser cache
-        const storedTime = localStorage.getItem(riskCheckLocalStorageKey)
-        const checkExpirationTime = storedTime ? parseInt(storedTime) : now - 1
-        if (checkExpirationTime < Date.now()) {
-          const headers = new Headers({ 'Content-Type': 'application/json' })
-          fetch('https://api.uniswap.org/v1/screen', {
-            method: 'POST',
-            headers,
-            body: JSON.stringify({ address: account }),
-          })
-            .then((res) => res.json())
-            .then((data) => {
-              if (data.block) {
-                dispatch(setOpenModal(ApplicationModal.BLOCKED_ACCOUNT))
-              }
-            })
-            .catch(() => {
-              dispatch(setOpenModal(null))
-            })
+    if (!account) return
+
+    // TODO: add back local browser cacheing (revisit 11/13/2023)
+    const headers = new Headers({ 'Content-Type': 'application/json' })
+    fetch('https://api.uniswap.org/v1/screen', {
+      method: 'POST',
+      headers,
+      body: JSON.stringify({ address: account }),
+    })
+      .then((res) => res.json())
+      .then((data) => {
+        if (data.block) {
+          dispatch(setOpenModal(ApplicationModal.BLOCKED_ACCOUNT))
         }
-      } finally {
-        // Set item to have 1 day local cache storage
-        localStorage.setItem(riskCheckLocalStorageKey, (now + ms(`1d`)).toString())
-      }
-    }
+      })
+      .catch(() => {
+        dispatch(setOpenModal(null))
+      })
   }, [account, dispatch])
 }

src/nft/utils/blocklist.ts

@@ -10,4 +10,5 @@ export const blocklistedCollections = [
   '0x8e52fb89b6311bd9ec36bd7cea9a0c311fd27a92',
   '0x2079c2765462af6d78a9ccbddb6ff3c6d4ba2e24',
   '0xd4d871419714b778ebec2e22c7c53572b573706e',
+  '0x383e2070a38f3205be0f3ea4fb05a5c13ade6cf4',
 ]

src/state/migrations.test.ts

@@ -13,7 +13,7 @@ const defaultState = {
   user: {},
   _persist: {
     rehydrated: true,
-    version: 2,
+    version: 4,
   },
   application: {
     chainId: null,

src/state/migrations.ts

@@ -4,6 +4,8 @@ import { MigrationConfig } from 'redux-persist/es/createMigrate'
 import { migration0 } from './migrations/0'
 import { migration1 } from './migrations/1'
 import { migration2 } from './migrations/2'
+import { migration3 } from './migrations/3'
+import { migration4 } from './migrations/4'
 import { legacyLocalStorageMigration } from './migrations/legacy'
 
 /**
@@ -19,6 +21,8 @@ export const migrations: MigrationManifest = {
   0: migration0,
   1: migration1,
   2: migration2,
+  3: migration3,
+  4: migration4,
 }
 
 // We use a custom migration function for the initial state, because redux-persist

src/state/migrations/3.ts

@@ -26,7 +26,7 @@ export const migration3 = (state: PersistAppStateV3 | undefined) => {
     }
     for (const [chainId, address] of Object.entries(USDCe_ADDRESSES)) {
       const chainIdKey = Number(chainId) as ChainId
-      if (state.user.tokens[chainIdKey]?.[address]) {
+      if (state.user.tokens?.[chainIdKey]?.[address]) {
         state.user.tokens[chainIdKey][address] = serializeToken(
           new Token(chainIdKey, address, 6, 'USDC.e', 'Bridged USDC')
         )
@@ -40,7 +40,7 @@ export const migration3 = (state: PersistAppStateV3 | undefined) => {
       'USDbC',
       'USD Base Coin'
     )
-    if (state.user.tokens[ChainId.BASE]?.[USDbC_BASE.address]) {
+    if (state.user.tokens?.[ChainId.BASE]?.[USDbC_BASE.address]) {
       state.user.tokens[ChainId.BASE][USDbC_BASE.address] = serializeToken(USDbC_BASE)
     }
     return {

src/state/migrations/4.test.ts

@@ -0,0 +1,46 @@
+import { DEFAULT_LOCALE } from 'constants/locales'
+import { createMigrate } from 'redux-persist'
+import { RouterPreference } from 'state/routing/types'
+import { SlippageTolerance } from 'state/user/types'
+
+import { migration1 } from './1'
+import { migration2 } from './2'
+import { migration3 } from './3'
+import { migration4, PersistAppStateV4 } from './4'
+
+const previousState: PersistAppStateV4 = {
+  user: {
+    userLocale: 'de-DE',
+    userRouterPreference: RouterPreference.API,
+    userHideClosedPositions: false,
+    userSlippageTolerance: SlippageTolerance.Auto,
+    userSlippageToleranceHasBeenMigratedToAuto: true,
+    userDeadline: 1800,
+    tokens: {},
+    pairs: {},
+    timestamp: Date.now(),
+    hideAndroidAnnouncementBanner: false,
+  },
+  _persist: {
+    version: 3,
+    rehydrated: true,
+  },
+}
+
+describe('migration to v4', () => {
+  it('should migrate users who currently have German as their set locale', async () => {
+    const migrator = createMigrate(
+      {
+        1: migration1,
+        2: migration2,
+        3: migration3,
+        4: migration4,
+      },
+      { debug: false }
+    )
+    const result: any = await migrator(previousState, 4)
+    expect(result.user.userLocale).toEqual(DEFAULT_LOCALE)
+
+    expect(result?._persist.version).toEqual(4)
+  })
+})

src/state/migrations/4.ts

@@ -0,0 +1,28 @@
+import { DEFAULT_LOCALE } from 'constants/locales'
+import { PersistState } from 'redux-persist'
+import { UserState } from 'state/user/reducer'
+
+export type PersistAppStateV4 = {
+  _persist: PersistState
+} & { user?: UserState }
+
+/**
+ * Migration to set german locale to default locale, after
+ * the german locale was removed from supported locales.
+ */
+export const migration4 = (state: PersistAppStateV4 | undefined) => {
+  if (state?.user) {
+    if (state.user.userLocale === 'de-DE') {
+      state.user.userLocale = DEFAULT_LOCALE
+    }
+
+    return {
+      ...state,
+      _persist: {
+        ...state._persist,
+        version: 4,
+      },
+    }
+  }
+  return state
+}

src/state/reducer.ts

@@ -44,7 +44,7 @@ export type AppState = ReturnType<typeof appReducer>
 
 const persistConfig: PersistConfig<AppState> = {
   key: 'interface',
-  version: 2, // see migrations.ts for more details about this version
+  version: 4, // see migrations.ts for more details about this version
   storage: localForage.createInstance({
     name: 'redux',
   }),

src/state/routing/gas.ts

@@ -1,5 +1,5 @@
 import { MaxUint256, PERMIT2_ADDRESS } from '@uniswap/permit2-sdk'
-import { Currency } from '@uniswap/sdk-core'
+import { ChainId, Currency } from '@uniswap/sdk-core'
 import ERC20_ABI from 'abis/erc20.json'
 import { Erc20, Weth } from 'abis/types'
 import WETH_ABI from 'abis/weth.json'
@@ -26,6 +26,12 @@ export async function getApproveInfo(
   // If any of these arguments aren't provided, then we cannot generate approval cost info
   if (!account || !usdCostPerGas) return { needsApprove: false }
 
+  // routing-api under estimates gas for Arbitrum swaps so it inflates cost per gas by a lot
+  // so disable showing approves for Arbitrum until routing-api gives more accurate gas estimates
+  if (currency.chainId === ChainId.ARBITRUM_ONE || currency.chainId === ChainId.ARBITRUM_GOERLI) {
+    return { needsApprove: false }
+  }
+
   const provider = DEPRECATED_RPC_PROVIDERS[currency.chainId as SupportedInterfaceChain]
   const tokenContract = getContract(currency.address, ERC20_ABI, provider) as Erc20
 

src/utils/nativeTokens.ts

@@ -1,4 +1,4 @@
-import { nativeOnChain } from 'constants/tokens'
+import { MATIC_POLYGON, nativeOnChain } from 'constants/tokens'
 import { Chain } from 'graphql/data/__generated__/types-and-hooks'
 import { supportedChainIdFromGQLChain } from 'graphql/data/util'
 
@@ -10,8 +10,11 @@ export function getNativeTokenDBAddress(chain: Chain): string | undefined {
   switch (chain) {
     // Celo & Polygon have precompiles for their native tokens
     case Chain.Celo:
-    case Chain.Polygon:
       return nativeOnChain(pageChainId).wrapped.address
+    case Chain.Polygon:
+      // Like Celo, native MATIC does have a ERC20 precompile, but we use WMATIC in routing/pools
+      // So instead of returning nativeOnChain().wrapped.address, should directly use the precompile address here
+      return MATIC_POLYGON.address
     case Chain.Ethereum:
     case Chain.Arbitrum:
     case Chain.EthereumGoerli: