1.34.1-0.1.0: [fix] network switch bug (#673)

* 1.34.1-0.1.0: [fix] Only show switch network modal once per wallet check
* Fix formatting

Author: taylorjdawson

package.json

@@ -1,6 +1,6 @@
 {
   "name": "bnc-onboard",
-  "version": "1.34.1",
+  "version": "1.34.1-0.1.0",
   "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

@@ -14,6 +14,19 @@ function network(
 ): (currentState: StateAndHelpers) => Promise<WalletCheckModal | undefined> {
   const { heading, description, icon, html, button } = options
 
+  let networkCheckRequested = false
+  let prevWalletCheckInProgressValue: boolean
+
+  app.subscribe(({ walletCheckInProgress }: AppState) => {
+    if (
+      prevWalletCheckInProgressValue === false &&
+      walletCheckInProgress === true
+    ) {
+      networkCheckRequested = false
+    }
+    prevWalletCheckInProgressValue = walletCheckInProgress
+  })
+
   return async (stateAndHelpers: StateAndHelpers) => {
     const {
       network,
@@ -42,8 +55,13 @@ function network(
       }
     }
     // Adds a check for WalletConnect since it hangs for unsupported rpc methods
-    if (getProviderName(wallet?.provider) !== 'WalletConnect') {
+    if (
+      !networkCheckRequested &&
+      stateStore.network.get() != appNetworkId &&
+      getProviderName(wallet?.provider) !== 'WalletConnect'
+    ) {
       try {
+        networkCheckRequested = true
         await wallet?.provider?.request({
           method: 'wallet_switchEthereumChain',
           params: [{ chainId: '0x' + appNetworkId?.toString(16) }]