Skip to content

Commit

Permalink
feat: turn on wcv2 wallet support (#5388)
Browse files Browse the repository at this point in the history
* feat: turn on wcv2 wallet support

* chore: remove "v2"
  • Loading branch information
0xApotheosis authored Oct 3, 2023
1 parent 4e0571c commit e4c190d
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .env.base
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ REACT_APP_FEATURE_NFT_METADATA=false
REACT_APP_FEATURE_CHATWOOT=false
REACT_APP_FEATURE_COINBASE_WALLET=true
REACT_APP_FEATURE_ADVANCED_SLIPPAGE=true
REACT_APP_FEATURE_WALLET_CONNECT_V2=false
REACT_APP_FEATURE_WALLET_CONNECT_V2=true
REACT_APP_FEATURE_FOX_DISCOUNTS=false

# absolute URL prefix
Expand Down
1 change: 0 additions & 1 deletion .env.dev
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
# feature flags
REACT_APP_FEATURE_NFT_METADATA=true
REACT_APP_FEATURE_WALLET_CONNECT_V2=true
REACT_APP_FEATURE_FOX_DISCOUNTS=true

# logging
Expand Down
1 change: 0 additions & 1 deletion .env.develop
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# feature flags
REACT_APP_FEATURE_NFT_METADATA=true
REACT_APP_FEATURE_CHATWOOT=true
REACT_APP_FEATURE_WALLET_CONNECT_V2=true

# mixpanel
REACT_APP_MIXPANEL_TOKEN=1c1369f6ea23a6404bac41b42817cc4b
Expand Down
2 changes: 1 addition & 1 deletion src/context/WalletProvider/WalletConnectV2/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ export const WalletConnectV2Config: Omit<SupportedWalletInfo, 'routes'> = {
adapters: [WalletConnectV2Adapter],
supportsMobile: 'both',
icon: WalletConnectIcon,
name: 'WalletConnectV2',
name: 'WalletConnect',
description: 'v2',
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ export const WalletConnectV2Provider: FC<PropsWithChildren> = ({ children }) =>

useEffect(() => {
// If we are connected to a wallet via wallet connect then this logic does not apply.
if (walletInfo?.name === 'WalletConnectV2') return
if (walletInfo?.name === 'WalletConnect') return

// NOTE: don't use `useAppSelector(selectWalletId)` because it introduces a race condition
const deviceId = walletInfo?.deviceId
Expand Down

0 comments on commit e4c190d

Please sign in to comment.