💂 ReadonlyNetworksProvider fix refreshing cycle (#843)

TrueFiEng · Jun 24, 2022 · d07e0df · d07e0df · github-actions · Jun 24, 2022
1 parent 06cac78
commit d07e0df
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 10 deletions.
diff --git a/.changeset/tiny-jars-tan.md b/.changeset/tiny-jars-tan.md
@@ -0,0 +1,5 @@
+---
+'@usedapp/core': patch
+---
+
+Fix refreshing cycle in ReadonlyNetworksProvider
diff --git a/packages/core/src/providers/network/readonlyNetworks/provider.tsx b/packages/core/src/providers/network/readonlyNetworks/provider.tsx
@@ -1,4 +1,4 @@
-import { ReactNode, useCallback, useEffect, useReducer, useState } from 'react'
+import { ReactNode, useCallback, useEffect, useMemo, useReducer, useState } from 'react'
 import { providers } from 'ethers'
 import { useConfig } from '../../../hooks'
 import { Providers } from './model'
@@ -70,14 +70,13 @@ export function ReadonlyNetworksProvider({ providerOverrides = {}, children }: N
     }
   }, [providers, getPollingInterval])
 
-  return (
-    <ReadonlyNetworksContext.Provider
-      value={{
-        providers,
-        updateNetworkState: dispatchNetworkState,
-      }}
-    >
-      {children}
-    </ReadonlyNetworksContext.Provider>
+  const networks = useMemo(
+    () => ({
+      providers,
+      updateNetworkState: dispatchNetworkState,
+    }),
+    [providers, dispatchNetworkState]
   )
+
+  return <ReadonlyNetworksContext.Provider value={networks}>{children}</ReadonlyNetworksContext.Provider>
 }