chore: revert path based routing (#7192)

* Revert "feat: use router depending on the origin (#6982)"

This reverts commit c9b4016b.

* updating tests

* fixing import from styled components

* fixing styled imports

README.md

@@ -69,10 +69,10 @@ Other things to note:
 
 The Uniswap Interface supports swapping, adding liquidity, removing liquidity and migrating liquidity for Uniswap protocol V2.
 
-- Swap on Uniswap V2: <https://app.uniswap.org/swap?use=v2>
-- View V2 liquidity: <https://app.uniswap.org/pools/v2>
-- Add V2 liquidity: <https://app.uniswap.org/add/v2>
-- Migrate V2 liquidity to V3: <https://app.uniswap.org/migrate/v2>
+- Swap on Uniswap V2: <https://app.uniswap.org/#/swap?use=v2>
+- View V2 liquidity: <https://app.uniswap.org/#/pools/v2>
+- Add V2 liquidity: <https://app.uniswap.org/#/add/v2>
+- Migrate V2 liquidity to V3: <https://app.uniswap.org/#/migrate/v2>
 
 ## Accessing Uniswap V1
 

cypress/e2e/nfts.test.ts

@@ -12,7 +12,7 @@ describe('Testing nfts', () => {
   })
 
   it('should load pudgy penguin collection page', () => {
-    cy.visit(`/nfts/collection/${PUDGY_COLLECTION_ADDRESS}`)
+    cy.visit(`/#/nfts/collection/${PUDGY_COLLECTION_ADDRESS}`)
     cy.get(getTestSelector('nft-collection-asset')).should('exist')
     cy.get(getTestSelector('nft-collection-filter-buy-now')).should('not.exist')
     cy.get(getTestSelector('nft-filter')).first().click()
@@ -20,13 +20,13 @@ describe('Testing nfts', () => {
   })
 
   it('should be able to navigate to activity', () => {
-    cy.visit(`/nfts/collection/${PUDGY_COLLECTION_ADDRESS}`)
+    cy.visit(`/#/nfts/collection/${PUDGY_COLLECTION_ADDRESS}`)
     cy.get(getTestSelector('nft-activity')).first().click()
     cy.get(getTestSelector('nft-activity-row')).should('exist')
   })
 
   it('should go to the details page', () => {
-    cy.visit(`/nfts/collection/${PUDGY_COLLECTION_ADDRESS}`)
+    cy.visit(`/#/nfts/collection/${PUDGY_COLLECTION_ADDRESS}`)
     cy.get(getTestSelector('nft-filter')).first().click()
     cy.get(getTestSelector('nft-collection-filter-buy-now')).click()
     cy.get(getTestSelector('nft-collection-asset')).first().click()
@@ -37,7 +37,7 @@ describe('Testing nfts', () => {
   })
 
   it('should toggle buy now on details page', () => {
-    cy.visit(`/nfts/collection/${PUDGY_COLLECTION_ADDRESS}`)
+    cy.visit(`/#/nfts/collection/${PUDGY_COLLECTION_ADDRESS}`)
     cy.get(getTestSelector('nft-filter')).first().click()
     cy.get(getTestSelector('nft-collection-filter-buy-now')).click()
     cy.get(getTestSelector('nft-collection-asset')).first().click()

cypress/e2e/token-explore.test.ts

@@ -69,5 +69,8 @@ describe('Token explore', () => {
     cy.get(getTestSelector('tokens-network-filter-selected')).click()
     cy.get(getTestSelector('tokens-network-filter-option-optimism')).click()
     cy.get(getTestSelector('tokens-network-filter-selected')).should('contain', 'Optimism')
+    cy.reload()
+    cy.get(getTestSelector('tokens-network-filter-selected')).should('contain', 'Optimism')
+    cy.get(getTestSelector('chain-selector-logo')).invoke('attr', 'alt').should('eq', 'Ethereum')
   })
 })

cypress/e2e/universal-search.test.ts

@@ -30,7 +30,7 @@ describe('Universal search bar', () => {
       .and('contain.text', '$')
       .and('contain.text', '%')
       .click()
-    cy.location('pathname').should('equal', '/tokens/ethereum/0x1f9840a85d5af5bf1d1762f925bdaddc4201f984')
+    cy.location('hash').should('equal', '#/tokens/ethereum/0x1f9840a85d5af5bf1d1762f925bdaddc4201f984')
 
     openSearch()
     cy.get(getTestSelector('searchbar-dropdown'))

cypress/e2e/wallet-dropdown.test.ts

@@ -32,11 +32,11 @@ describe('Wallet Dropdown', () => {
       }
 
       cy.get(getTestSelector('wallet-language-item')).contains('Afrikaans').click({ force: true })
-      cy.location('search').should('match', /\?lng=af-ZA$/)
+      cy.location('hash').should('match', /\?lng=af-ZA$/)
       cy.contains('Uniswap available in: English')
 
       cy.get(getTestSelector('wallet-language-item')).contains('English').click({ force: true })
-      cy.location('search').should('match', /\?lng=en-US$/)
+      cy.location('hash').should('match', /\?lng=en-US$/)
       cy.contains('Uniswap available in: English').should('not.exist')
     })
   }

cypress/staging/t9n.test.ts

@@ -12,7 +12,7 @@ describe('translations', () => {
     cy.get(getTestSelector('web3-status-connected')).click()
     cy.get(getTestSelector('wallet-settings')).click()
     cy.get(getTestSelector('wallet-language-item')).contains('français').click({ force: true })
-    cy.location('search').should('match', /\?lng=fr-FR$/)
+    cy.location('hash').should('match', /\?lng=fr-FR$/)
     cy.contains('Échanger')
     cy.contains('Uniswap disponible en : English')
   })

package.json

@@ -105,6 +105,7 @@
     "@types/react": "^18.0.15",
     "@types/react-dom": "^18.0.6",
     "@types/react-redux": "^7.1.24",
+    "@types/react-router-dom": "^5.3.3",
     "@types/react-table": "^7.7.12",
     "@types/react-virtualized-auto-sizer": "^1.0.0",
     "@types/react-window": "^1.8.2",

src/components/FiatOnrampModal/index.tsx

 import { Trans } from '@lingui/macro'
 import { useWeb3React } from '@web3-react/core'
 import { useCallback, useEffect, useState } from 'react'
-import { useHref } from 'react-router-dom'
 import { useCloseModal, useModalIsOpen } from 'state/application/hooks'
 import { ApplicationModal } from 'state/application/reducer'
 import styled, { useTheme } from 'styled-components'
@@ -80,8 +79,6 @@ export default function FiatOnrampModal() {
   const [error, setError] = useState<string | null>(null)
   const [loading, setLoading] = useState(false)
 
-  const swapUrl = useHref('/swap')
-
   const fetchSignedIframeUrl = useCallback(async () => {
     if (!account) {
       setError('Please connect an account before making a purchase.')
@@ -101,7 +98,7 @@ export default function FiatOnrampModal() {
           theme: isDarkMode ? 'dark' : 'light',
           colorCode: theme.accentAction,
           defaultCurrencyCode: 'eth',
-          redirectUrl: swapUrl,
+          redirectUrl: 'https://app.uniswap.org/#/swap',
           walletAddresses: JSON.stringify(
             MOONPAY_SUPPORTED_CURRENCY_CODES.reduce(
               (acc, currencyCode) => ({
@@ -121,7 +118,7 @@ export default function FiatOnrampModal() {
     } finally {
       setLoading(false)
     }
-  }, [account, isDarkMode, swapUrl, theme.accentAction])
+  }, [account, isDarkMode, theme.accentAction])
 
   useEffect(() => {
     fetchSignedIframeUrl()

src/components/Tokens/TokenTable/__snapshots__/TokenRow.test.tsx.snap

@@ -343,7 +343,7 @@ exports[`LoadedRow.tsx renders a row 1`] = `
   >
     <a
       class="c0"
-      href="/tokens/ethereum/0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48"
+      href="#/tokens/ethereum/0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48"
     >
       <div
         class="c1"

src/index.tsx

@@ -12,9 +12,8 @@ import { StrictMode } from 'react'
 import { createRoot } from 'react-dom/client'
 import { QueryClient, QueryClientProvider } from 'react-query'
 import { Provider } from 'react-redux'
-import { BrowserRouter, HashRouter } from 'react-router-dom'
+import { HashRouter } from 'react-router-dom'
 import { SystemThemeUpdater } from 'theme/components/ThemeToggle'
-import { isBrowserRouterEnabled } from 'utils/env'
 
 import Web3Provider from './components/Web3Provider'
 import { LanguageProvider } from './i18n'
@@ -52,14 +51,12 @@ const queryClient = new QueryClient()
 
 const container = document.getElementById('root') as HTMLElement
 
-const Router = isBrowserRouterEnabled() ? BrowserRouter : HashRouter
-
 createRoot(container).render(
   <StrictMode>
     <Provider store={store}>
       <FeatureFlagsProvider>
         <QueryClientProvider client={queryClient}>
-          <Router>
+          <HashRouter>
             <LanguageProvider>
               <Web3Provider>
                 <ApolloProvider client={apolloClient}>
@@ -73,7 +70,7 @@ createRoot(container).render(
                 </ApolloProvider>
               </Web3Provider>
             </LanguageProvider>
-          </Router>
+          </HashRouter>
         </QueryClientProvider>
       </FeatureFlagsProvider>
     </Provider>

src/nft/components/card/containers.tsx

@@ -95,7 +95,7 @@ const ActionButton = ({
     <StyledActionButton
       selected={isSelected}
       isDisabled={isDisabled}
-      onClick={(e) => (isDisabled ? undefined : clickActionButton(e))}
+      onClick={(e: React.MouseEvent) => (isDisabled ? undefined : clickActionButton(e))}
     >
       {children}
     </StyledActionButton>

src/nft/components/collection/Activity.css.ts

@@ -38,7 +38,6 @@ export const eventRow = style([
     borderBottomColor: 'backgroundOutline',
   }),
   {
-    textDecoration: 'none',
     height: '84px',
     ':hover': {
       background: themeVars.colors.backgroundSurface,

src/nft/components/collection/Activity.tsx

@@ -5,11 +5,10 @@ import { Box } from 'nft/components/Box'
 import { Column, Row } from 'nft/components/Flex'
 import { themeVars, vars } from 'nft/css/sprinkles.css'
 import { useBag, useIsMobile } from 'nft/hooks'
-import { ActivityEventType } from 'nft/types'
+import { ActivityEvent, ActivityEventType } from 'nft/types'
 import { fetchPrice } from 'nft/utils/fetchPrice'
 import { useCallback, useEffect, useReducer, useState } from 'react'
 import InfiniteScroll from 'react-infinite-scroll-component'
-import { Link } from 'react-router-dom'
 import styled from 'styled-components'
 import { useIsDarkMode } from 'theme/components/ThemeToggle'
 
@@ -61,6 +60,8 @@ export const reduceFilters = (state: typeof initialFilterState, action: { eventT
   return { ...state, [action.eventType]: !state[action.eventType] }
 }
 
+const baseHref = (event: ActivityEvent) => `/#/nfts/asset/${event.collectionAddress}/${event.tokenId}?origin=activity`
+
 export const Activity = ({ contractAddress, rarityVerified, collectionName, chainId }: ActivityProps) => {
   const [activeFilters, filtersDispatch] = useReducer(reduceFilters, initialFilterState)
 
@@ -137,11 +138,9 @@ export const Activity = ({ contractAddress, rarityVerified, collectionName, chai
                 (event, i) =>
                   event.eventType && (
                     <Box
-                      as={Link}
+                      as="a"
                       data-testid="nft-activity-row"
-                      // @ts-ignore Box component is not typed properly to typecheck
-                      // custom components' props and will incorrectly report `to` as invalid
-                      to={`/nfts/asset/${event.collectionAddress}/${event.tokenId}?origin=activity`}
+                      href={baseHref(event)}
                       className={styles.eventRow}
                       key={i}
                     >

src/nft/components/collection/__snapshots__/CollectionAsset.test.tsx.snap

@@ -309,7 +309,7 @@ exports[`NftCard renders correctly 1`] = `
   >
     <a
       class="c1"
-      href="/nfts/asset/0xed5af388653567af2f388e6224dc7c4b3241c544/3318?origin=collection"
+      href="#/nfts/asset/0xed5af388653567af2f388e6224dc7c4b3241c544/3318?origin=collection"
     >
       <div
         class="c2"

src/pages/App.tsx

@@ -106,14 +106,6 @@ const LazyLoadSpinner = () => (
   </SpinnerSVG>
 )
 
-const RedirectHashToPath = ({ children }: { children: JSX.Element }) => {
-  const { hash } = useLocation()
-  if (hash) {
-    return <Navigate to={hash.replace('#', '')} replace />
-  }
-  return children
-}
-
 export default function App() {
   const isLoaded = useFeatureFlagsIsLoaded()
   const [shouldDisableNFTRoutes, setShouldDisableNFTRoutes] = useAtom(shouldDisableNFTRoutesAtom)
@@ -217,16 +209,7 @@ export default function App() {
             <Suspense fallback={<Loader />}>
               {isLoaded ? (
                 <Routes>
-                  <Route
-                    path="/"
-                    element={
-                      // if react-router-dom matches "/" with window.location.hash defined, it means that we're
-                      // using BrowserRouter and can safely redirect to a path route
-                      <RedirectHashToPath>
-                        <Landing />
-                      </RedirectHashToPath>
-                    }
-                  />
+                  <Route path="/" element={<Landing />} />
 
                   <Route path="tokens" element={<Tokens />}>
                     <Route path=":chainName" />

src/pages/Landing/__snapshots__/index.test.tsx.snap

@@ -1741,7 +1741,7 @@ exports[`disable nft on landing page does not render nft information and card 1`
     >
       <a
         class="c2"
-        href="/swap"
+        href="#/swap"
       >
         <div
           class="c3"
@@ -2031,7 +2031,7 @@ exports[`disable nft on landing page does not render nft information and card 1`
       >
         <a
           class="c2 c25 c56 c57"
-          href="/swap"
+          href="#/swap"
         >
           <p
             class="c58"
@@ -2098,7 +2098,7 @@ exports[`disable nft on landing page does not render nft information and card 1`
       >
         <a
           class="c64"
-          href="/swap"
+          href="#/swap"
         >
           <div
             class="c65"
@@ -2178,7 +2178,7 @@ exports[`disable nft on landing page does not render nft information and card 1`
         </a>
         <a
           class="c70"
-          href="/pools"
+          href="#/pools"
         >
           <div
             class="c65"
@@ -2363,19 +2363,19 @@ exports[`disable nft on landing page does not render nft information and card 1`
             </span>
             <a
               class="c94 c95"
-              href="/swap"
+              href="#/swap"
             >
               Swap
             </a>
             <a
               class="c94 c95"
-              href="/tokens"
+              href="#/tokens"
             >
               Tokens
             </a>
             <a
               class="c94 c95"
-              href="/pools"
+              href="#/pools"
             >
               Pools
             </a>
@@ -4315,7 +4315,7 @@ exports[`disable nft on landing page renders nft information and card 1`] = `
     >
       <a
         class="c2"
-        href="/swap"
+        href="#/swap"
       >
         <div
           class="c3"
@@ -4605,7 +4605,7 @@ exports[`disable nft on landing page renders nft information and card 1`] = `
       >
         <a
           class="c2 c25 c56 c57"
-          href="/swap"
+          href="#/swap"
         >
           <p
             class="c58"
@@ -4672,7 +4672,7 @@ exports[`disable nft on landing page renders nft information and card 1`] = `
       >
         <a
           class="c64"
-          href="/swap"
+          href="#/swap"
         >
           <div
             class="c65"
@@ -4698,7 +4698,7 @@ exports[`disable nft on landing page renders nft information and card 1`] = `
         </a>
         <a
           class="c69"
-          href="/nfts"
+          href="#/nfts"
         >
           <div
             class="c65"
@@ -4778,7 +4778,7 @@ exports[`disable nft on landing page renders nft information and card 1`] = `
         </a>
         <a
           class="c71"
-          href="/pools"
+          href="#/pools"
         >
           <div
             class="c65"
@@ -4963,25 +4963,25 @@ exports[`disable nft on landing page renders nft information and card 1`] = `
             </span>
             <a
               class="c95 c96"
-              href="/swap"
+              href="#/swap"
             >
               Swap
             </a>
             <a
               class="c95 c96"
-              href="/tokens"
+              href="#/tokens"
             >
               Tokens
             </a>
             <a
               class="c95 c96"
-              href="/nfts"
+              href="#/nfts"
             >
               NFTs
             </a>
             <a
               class="c95 c96"
-              href="/pools"
+              href="#/pools"
             >
               Pools
             </a>

src/pages/Pool/PositionPage.tsx

@@ -2,7 +2,7 @@ import { BigNumber } from '@ethersproject/bignumber'
 import type { TransactionResponse } from '@ethersproject/providers'
 import { Trans } from '@lingui/macro'
 import { InterfacePageName } from '@uniswap/analytics-events'
-import { Currency, CurrencyAmount, Fraction, Percent, Price, Token } from '@uniswap/sdk-core'
+import { ChainId, Currency, CurrencyAmount, Fraction, Percent, Price, Token } from '@uniswap/sdk-core'
 import { NonfungiblePositionManager, Pool, Position } from '@uniswap/v3-sdk'
 import { useWeb3React } from '@web3-react/core'
 import { Trace } from 'analytics'
@@ -29,12 +29,12 @@ import { useV3PositionFees } from 'hooks/useV3PositionFees'
 import { useV3PositionFromTokenId } from 'hooks/useV3Positions'
 import { useSingleCallResult } from 'lib/hooks/multicall'
 import useNativeCurrency from 'lib/hooks/useNativeCurrency'
-import { PropsWithChildren, useCallback, useMemo, useRef, useState } from 'react'
+import { useCallback, useMemo, useRef, useState } from 'react'
 import { Link, useParams } from 'react-router-dom'
 import { Bound } from 'state/mint/v3/actions'
 import { useIsTransactionPending, useTransactionAdder } from 'state/transactions/hooks'
 import styled, { useTheme } from 'styled-components'
-import { ExternalLink, HideExtraSmall, HideSmall, StyledRouterLink, ThemedText } from 'theme'
+import { ExternalLink, HideExtraSmall, HideSmall, ThemedText } from 'theme'
 import { currencyId } from 'utils/currencyId'
 import { formatCurrencyAmount } from 'utils/formatCurrencyAmount'
 import { formatPrice, NumberType } from 'utils/formatNumbers'
@@ -52,6 +52,15 @@ import { calculateGasMargin } from '../../utils/calculateGasMargin'
 import { ExplorerDataType, getExplorerLink } from '../../utils/getExplorerLink'
 import { LoadingRows } from './styled'
 
+const getTokenLink = (chainId: ChainId, address: string) => {
+  if (isGqlSupportedChain(chainId)) {
+    const chainName = CHAIN_IDS_TO_NAMES[chainId]
+    return `${window.location.origin}/#/tokens/${chainName}/${address}`
+  } else {
+    return getExplorerLink(chainId, address, ExplorerDataType.TOKEN)
+  }
+}
+
 const PositionPageButtonPrimary = styled(ButtonPrimary)`
   width: 228px;
   height: 40px;
@@ -201,31 +210,17 @@ function CurrentPriceCard({
   )
 }
 
-const TokenLink = ({
-  children,
-  chainId,
-  address,
-}: PropsWithChildren<{ chainId: keyof typeof CHAIN_IDS_TO_NAMES; address: string }>) => {
-  const chainName = CHAIN_IDS_TO_NAMES[chainId]
-  return <StyledRouterLink to={`/tokens/${chainName}/${address}`}>{children}</StyledRouterLink>
-}
-
-const ExternalTokenLink = ({ children, chainId, address }: PropsWithChildren<{ chainId: number; address: string }>) => {
-  return <ExternalLink href={getExplorerLink(chainId, address, ExplorerDataType.TOKEN)}>{children}</ExternalLink>
-}
-
 function LinkedCurrency({ chainId, currency }: { chainId?: number; currency?: Currency }) {
   const address = (currency as Token)?.address
 
   if (typeof chainId === 'number' && address) {
-    const Link = isGqlSupportedChain(chainId) ? TokenLink : ExternalTokenLink
     return (
-      <Link chainId={chainId} address={address}>
+      <ExternalLink href={getTokenLink(chainId, address)}>
         <RowFixed>
           <CurrencyLogo currency={currency} size="20px" style={{ marginRight: '0.5rem' }} />
           <ThemedText.DeprecatedMain>{currency?.symbol} ↗</ThemedText.DeprecatedMain>
         </RowFixed>
-      </Link>
+      </ExternalLink>
     )
   }
 

src/serviceWorker/document.test.ts

@@ -17,15 +17,15 @@ describe('document', () => {
       [{ request: { mode: 'navigate' }, url: { hostname: 'example.com', pathname: '' } }, false],
       [{ request: {}, url: { hostname: 'app.uniswap.org', pathname: '' } }, false],
       [{ request: { mode: 'navigate' }, url: { hostname: 'app.uniswap.org', pathname: '' } }, true],
-      [{ request: { mode: 'navigate' }, url: { hostname: 'app.uniswap.org', pathname: '/swap' } }, true],
+      [{ request: { mode: 'navigate' }, url: { hostname: 'app.uniswap.org', pathname: '/#/swap' } }, true],
       [{ request: { mode: 'navigate' }, url: { hostname: 'app.uniswap.org', pathname: '/asset.gif' } }, false],
       [{ request: {}, url: { hostname: 'app.uniswap-staging.org', pathname: '' } }, false],
       [{ request: { mode: 'navigate' }, url: { hostname: 'app.uniswap-staging.org', pathname: '' } }, true],
-      [{ request: { mode: 'navigate' }, url: { hostname: 'app.uniswap-staging.org', pathname: '/swap' } }, true],
+      [{ request: { mode: 'navigate' }, url: { hostname: 'app.uniswap-staging.org', pathname: '/#/swap' } }, true],
       [{ request: { mode: 'navigate' }, url: { hostname: 'app.uniswap-staging.org', pathname: '/asset.gif' } }, false],
       [{ request: {}, url: { hostname: 'localhost', pathname: '' } }, false],
       [{ request: { mode: 'navigate' }, url: { hostname: 'localhost', pathname: '' } }, true],
-      [{ request: { mode: 'navigate' }, url: { hostname: 'localhost', pathname: '/swap' } }, true],
+      [{ request: { mode: 'navigate' }, url: { hostname: 'localhost', pathname: '/#/swap' } }, true],
       [{ request: { mode: 'navigate' }, url: { hostname: 'localhost', pathname: '/asset.gif' } }, false],
     ] as [RouteMatchCallbackOptions, boolean][]
 

src/test-utils/render.tsx

@@ -10,7 +10,7 @@ import { en } from 'make-plural/plurals'
 import { ReactElement, ReactNode } from 'react'
 import { QueryClient, QueryClientProvider } from 'react-query'
 import { Provider } from 'react-redux'
-import { BrowserRouter } from 'react-router-dom'
+import { HashRouter } from 'react-router-dom'
 import store from 'state'
 import ThemeProvider from 'theme'
 
@@ -30,7 +30,7 @@ const WithProviders = ({ children }: { children?: ReactNode }) => {
     <MockedI18nProvider>
       <Provider store={store}>
         <QueryClientProvider client={queryClient}>
-          <BrowserRouter>
+          <HashRouter>
             {/*
              * Web3Provider is mocked through setupTests.ts
              * To test behavior that depends on Web3Provider, use jest.unmock('@web3-react/core')
@@ -40,7 +40,7 @@ const WithProviders = ({ children }: { children?: ReactNode }) => {
                 <ThemeProvider>{children}</ThemeProvider>
               </BlockNumberProvider>
             </MockedProvider>
-          </BrowserRouter>
+          </HashRouter>
         </QueryClientProvider>
       </Provider>
     </MockedI18nProvider>

src/tracing/trace.test.ts

@@ -103,7 +103,7 @@ describe('trace', () => {
       const errorEvent: ErrorEvent = {
         type: undefined,
         request: {
-          url: 'https://app.uniswap.org/pools',
+          url: 'https://app.uniswap.org/#/pools',
         },
       }
       const eventHint: EventHint = {}
@@ -114,7 +114,7 @@ describe('trace', () => {
       const errorEvent: ErrorEvent = {
         type: undefined,
         request: {
-          url: 'https://app.uniswap.org/',
+          url: 'https://app.uniswap.org/#',
         },
       }
       const eventHint: EventHint = {}

src/utils/env.ts

@@ -23,25 +23,6 @@ export function isAppUniswapStagingOrg({ hostname }: { hostname: string }): bool
   return hostname === 'app.uniswap-staging.org'
 }
 
-export function isBrowserRouterEnabled(): boolean {
-  if (isProductionEnv()) {
-    if (
-      isAppUniswapOrg(window.location) ||
-      isAppUniswapStagingOrg(window.location) ||
-      // Cypress tests
-      isLocalhost(window.location)
-    ) {
-      return true
-    }
-    return false
-  }
-  return true
-}
-
-function isLocalhost({ hostname }: { hostname: string }): boolean {
-  return hostname === 'localhost'
-}
-
 export function isSentryEnabled(): boolean {
   // Disable in e2e test environments
   if (isStagingEnv() && !isAppUniswapStagingOrg(window.location)) return false

yarn.lock

@@ -5635,6 +5635,11 @@
   dependencies:
     "@types/node" "*"
 
+"@types/history@*", "@types/history@^4.7.11":
+  version "4.7.11"
+  resolved "https://registry.yarnpkg.com/@types/history/-/history-4.7.11.tgz#56588b17ae8f50c53983a524fc3cc47437969d64"
+  integrity sha512-qjDJRrmvBMiTx+jyLxvLfJU7UznFuokDv4f3WRuriHKERccVpFU+8XMQUAbDzoiJCsmexxRExQeMwwCdamSKDA==
+
 "@types/hoist-non-react-statics@*", "@types/hoist-non-react-statics@^3.3.0", "@types/hoist-non-react-statics@^3.3.1":
   version "3.3.1"
   resolved "https://registry.yarnpkg.com/@types/hoist-non-react-statics/-/hoist-non-react-statics-3.3.1.tgz#1124aafe5118cb591977aeb1ceaaed1070eb039f"
@@ -5837,6 +5842,23 @@
     hoist-non-react-statics "^3.3.0"
     redux "^4.0.0"
 
+"@types/react-router-dom@^5.3.3":
+  version "5.3.3"
+  resolved "https://registry.yarnpkg.com/@types/react-router-dom/-/react-router-dom-5.3.3.tgz#e9d6b4a66fcdbd651a5f106c2656a30088cc1e83"
+  integrity sha512-kpqnYK4wcdm5UaWI3fLcELopqLrHgLqNsdpHauzlQktfkHL3npOSwtj1Uz9oKBAzs7lFtVkV8j83voAz2D8fhw==
+  dependencies:
+    "@types/history" "^4.7.11"
+    "@types/react" "*"
+    "@types/react-router" "*"
+
+"@types/react-router@*":
+  version "5.1.16"
+  resolved "https://registry.yarnpkg.com/@types/react-router/-/react-router-5.1.16.tgz#f3ba045fb96634e38b21531c482f9aeb37608a99"
+  integrity sha512-8d7nR/fNSqlTFGHti0R3F9WwIertOaaA1UEB8/jr5l5mDMOs4CidEgvvYMw4ivqrBK+vtVLxyTj2P+Pr/dtgzg==
+  dependencies:
+    "@types/history" "*"
+    "@types/react" "*"
+
 "@types/react-table@^7.7.12":
   version "7.7.12"
   resolved "https://registry.yarnpkg.com/@types/react-table/-/react-table-7.7.12.tgz#628011d3cb695b07c678704a61f2f1d5b8e567fd"