fix: Adjust safe creation event to include safeAddress

safe-global · Oct 18, 2023 · bf16b73 · bf16b73
1 parent 8189cd7
commit bf16b73
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/components/new-safe/create/steps/StatusStep/useSafeCreationEffects.ts b/src/components/new-safe/create/steps/StatusStep/useSafeCreationEffects.ts
@@ -7,6 +7,7 @@ import { updateAddressBook } from '@/components/new-safe/create/logic/address-bo
 import { useAppDispatch } from '@/store'
 import useChainId from '@/hooks/useChainId'
 import { usePendingSafe } from './usePendingSafe'
+import { gtmSetSafeAddress } from '@/services/analytics/gtm'
 
 const useSafeCreationEffects = ({
   status,
@@ -74,6 +75,7 @@ const useSafeCreationEffects = ({
   // Tracking
   useEffect(() => {
     if (status === SafeCreationStatus.SUCCESS) {
+      pendingSafe?.safeAddress && gtmSetSafeAddress(pendingSafe.safeAddress)
       trackEvent(CREATE_SAFE_EVENTS.CREATED_SAFE)
       return
     }
@@ -82,7 +84,7 @@ const useSafeCreationEffects = ({
       trackEvent(CREATE_SAFE_EVENTS.REJECT_CREATE_SAFE)
       return
     }
-  }, [status])
+  }, [pendingSafe?.safeAddress, status])
 }
 
 export default useSafeCreationEffects
diff --git a/src/services/analytics/useGtm.ts b/src/services/analytics/useGtm.ts
@@ -68,6 +68,8 @@ const useGtm = () => {
 
   // Set safe address for all GTM events
   useEffect(() => {
+    if (safeAddress === '') return
+
     gtmSetSafeAddress(safeAddress)
   }, [safeAddress])