1.33.0-0.0.1: [fix] Fixes walletconnect hanging on network check (#653)

- Fixes walletconnect hanging on network check

Author: taylorjdawson

package.json

@@ -1,6 +1,6 @@
 {
   "name": "bnc-onboard",
-  "version": "1.33.0",
+  "version": "1.33.0-0.0.1",
   "description": "Onboard users to web3 by allowing them to select a wallet, get that wallet ready to transact and have access to synced wallet state.",
   "keywords": [
     "ethereum",

src/modules/check/network.ts

@@ -1,4 +1,4 @@
-import { networkName } from '../../utilities'
+import { getProviderName, networkName } from '../../utilities'
 import {
   WalletCheckModal,
   StateAndHelpers,
@@ -41,14 +41,18 @@ function network(
         })
       }
     }
-    try {
-      await wallet?.provider?.request({
-        method: 'wallet_switchEthereumChain',
-        params: [{ chainId: '0x' + appNetworkId?.toString(16) }]
-      })
-    } catch (e) {
-      // Could not switch networks so proceed as normal through the checks
+    // Adds a check for WalletConnect since it hangs for unsupported rpc methods
+    if (getProviderName(wallet?.provider) !== 'WalletConnect') {
+      try {
+        await wallet?.provider?.request({
+          method: 'wallet_switchEthereumChain',
+          params: [{ chainId: '0x' + appNetworkId?.toString(16) }]
+        })
+      } catch (e) {
+        // Could not switch networks so proceed as normal through the checks
+      }
     }
+
     if (stateStore.network.get() != appNetworkId) {
       return {
         heading: heading || 'You Must Change Networks',