Make sure action event is tracked when handleAction is called

initState can be called from other places, so it's not the right place to send the event. COAND-845
Adyen · May 24, 2024 · b6d5936 · b6d5936
1 parent 220f737
commit b6d5936
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/await/src/main/java/com/adyen/checkout/await/internal/ui/DefaultAwaitDelegate.kt b/await/src/main/java/com/adyen/checkout/await/internal/ui/DefaultAwaitDelegate.kt
@@ -124,6 +124,12 @@ internal class DefaultAwaitDelegate(
 
         this.action = action
 
+        val event = GenericEvents.action(
+            component = action.paymentMethodType.orEmpty(),
+            subType = action.type.orEmpty(),
+        )
+        analyticsManager?.trackEvent(event)
+
         initState(action)
     }
 
@@ -137,12 +143,6 @@ internal class DefaultAwaitDelegate(
         }
         createOutputData(null, action)
 
-        val event = GenericEvents.action(
-            component = action.paymentMethodType.orEmpty(),
-            subType = action.type.orEmpty(),
-        )
-        analyticsManager?.trackEvent(event)
-
         startStatusPolling(paymentData, action)
     }
 

diff --git a/voucher/src/main/java/com/adyen/checkout/voucher/internal/ui/DefaultVoucherDelegate.kt b/voucher/src/main/java/com/adyen/checkout/voucher/internal/ui/DefaultVoucherDelegate.kt
@@ -120,6 +120,12 @@ internal class DefaultVoucherDelegate(
 
         this.action = action
 
+        val event = GenericEvents.action(
+            component = action.paymentMethodType.orEmpty(),
+            subType = action.type.orEmpty(),
+        )
+        analyticsManager?.trackEvent(event)
+
         initState(action)
     }
 
@@ -130,12 +136,6 @@ internal class DefaultVoucherDelegate(
             return
         }
 
-        val event = GenericEvents.action(
-            component = action.paymentMethodType.orEmpty(),
-            subType = action.type.orEmpty(),
-        )
-        analyticsManager?.trackEvent(event)
-
         _viewFlow.tryEmit(config.viewType)
 
         createOutputData(action, config)