fix: walletType could be undefined during onboarding, import

rainbow-me · Nov 8, 2024 · 4832161 · 4832161
1 parent 510a7be
commit 4832161
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/analytics/utils.ts b/src/analytics/utils.ts
@@ -94,6 +94,7 @@ export async function getWalletContext(address: Address): Promise<WalletContext>
   // currentAddressStore address is initialized to ''
   if (!address || address === ('' as Address)) return {};
 
+  // walletType maybe undefined after initial wallet creation
   const { wallets } = store.getState();
   const wallet = findWalletWithAccount(wallets.wallets || {}, address);
 

diff --git a/src/hooks/useInitializeWallet.ts b/src/hooks/useInitializeWallet.ts
@@ -87,6 +87,7 @@ export default function useInitializeWallet() {
         });
 
         // Capture wallet context in telemetry
+        // walletType maybe undefied after initial wallet creation
         const { walletType, walletAddressHash } = await getWalletContext(walletAddress as Address);
         const [deviceId] = await getOrCreateDeviceId();
 
@@ -97,7 +98,7 @@ export default function useInitializeWallet() {
         });
 
         // Allows calling telemetry before currentAddress is available (i.e. onboarding)
-        if (walletType && walletAddressHash) analyticsV2.setWalletContext({ walletAddressHash, walletType });
+        if (walletType || walletAddressHash) analyticsV2.setWalletContext({ walletAddressHash, walletType });
         analyticsV2.setDeviceId(deviceId);
         analyticsV2.identify();