refactor: connect with walletconnect

lightningminers · Dec 12, 2021 · c0638e0 · c0638e0
1 parent c390926
commit c0638e0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/src/connectors/walletConnect.ts b/src/connectors/walletConnect.ts
@@ -40,8 +40,8 @@ export class WalletConnectConnector extends BaseConnector {
 
   async connect() {
     try {
-      if (!this._provider)
-        this._provider = new WalletConnectProvider(this._options)
+      // Use new provider instance for every connect
+      this._provider = new WalletConnectProvider(this._options)
 
       this._provider.on('accountsChanged', this.onAccountsChanged)
       this._provider.on('chainChanged', this.onChainChanged)

diff --git a/src/hooks/useConnect.ts b/src/hooks/useConnect.ts
@@ -1,6 +1,6 @@
 import * as React from 'react'
 
-import { Connector, WalletConnectConnector } from '../connectors'
+import { Connector } from '../connectors'
 import { useContext } from './useContext'
 
 type State = {
@@ -23,20 +23,14 @@ export const useConnect = () => {
         const activeConnector = globalState?.connector
         if (connector === activeConnector) return
 
-        // Manually reset connector if user already tried WalletConnect
-        if (
-          connector instanceof WalletConnectConnector &&
-          connector.provider?.isWalletConnect
-        )
-          connector.provider = undefined
-
         setState((x) => ({
           ...x,
           loading: true,
           connector,
           error: undefined,
         }))
         const data = await connector.connect()
+
         // Update connector globally only after successful connection
         setGlobalState((x) => ({ ...x, connector, data }))
         setLastUsedConnector(connector.name)