fix: appcontext isSnapInstalled checks (#7808)

* fix: appcontext isSnapInstalled checks * feat: tackle last occurence
shapeshift · Sep 25, 2024 · b1bbb76 · b1bbb76
1 parent a6d3a38
commit b1bbb76
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/src/context/AppProvider/AppContext.tsx b/src/context/AppProvider/AppContext.tsx
@@ -63,7 +63,7 @@ export const AppProvider = ({ children }: { children: React.ReactNode }) => {
   const portfolioLoadingStatus = useSelector(selectPortfolioLoadingStatus)
   const portfolioAssetIds = useSelector(selectPortfolioAssetIds)
   const routeAssetId = useRouteAssetId()
-  const isSnapInstalled = Boolean(useIsSnapInstalled())
+  const { isSnapInstalled } = useIsSnapInstalled()
   const previousIsSnapInstalled = usePrevious(isSnapInstalled)
   const { close: closeModal, open: openModal } = useModal('ledgerOpenApp')
 
@@ -180,7 +180,12 @@ export const AppProvider = ({ children }: { children: React.ReactNode }) => {
           )
             break
 
-          const input = { accountNumber, chainIds, wallet, isSnapInstalled }
+          const input = {
+            accountNumber,
+            chainIds,
+            wallet,
+            isSnapInstalled: Boolean(isSnapInstalled),
+          }
           const accountIdsAndMetadata = await deriveAccountIdsAndMetadata(input)
           const accountIds = Object.keys(accountIdsAndMetadata)
 

diff --git a/src/pages/Accounts/AddAccountModal.tsx b/src/pages/Accounts/AddAccountModal.tsx
@@ -60,7 +60,7 @@ export const AddAccountModal = () => {
     selectMaybeNextAccountNumberByChainId(s, filter),
   )
 
-  const isSnapInstalled = Boolean(useIsSnapInstalled())
+  const { isSnapInstalled } = useIsSnapInstalled()
 
   const isMetaMaskMultichainWallet = wallet instanceof MetaMaskShapeShiftMultiChainHDWallet
   const unsupportedSnapChainIds = useMemo(() => {
@@ -73,7 +73,7 @@ export const AddAccountModal = () => {
           accountNumber: nextAccountNumber,
           chainId,
           wallet,
-          isSnapInstalled,
+          isSnapInstalled: Boolean(isSnapInstalled),
         }),
     )
   }, [chainIds, isMetaMaskMultichainWallet, isSnapInstalled, nextAccountNumber, wallet])
@@ -105,7 +105,7 @@ export const AddAccountModal = () => {
         accountNumber,
         chainIds,
         wallet,
-        isSnapInstalled,
+        isSnapInstalled: Boolean(isSnapInstalled),
       })
 
       const { getAccount } = portfolioApi.endpoints