fix(capture-sdk): Fix sending returnAssisstantEnabled from configurat…

…ion endpoint PP-596
gini · Jul 12, 2024 · 147f994 · 147f994
1 parent 588f65f
commit 147f994
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 7 deletions.
diff --git a/bank-sdk/sdk/src/main/java/net/gini/android/bank/sdk/capture/CaptureFlowFragment.kt b/bank-sdk/sdk/src/main/java/net/gini/android/bank/sdk/capture/CaptureFlowFragment.kt
@@ -50,12 +50,9 @@ class CaptureFlowFragment(private val openWithDocument: Document? = null) :
 
     private val userAnalyticsEventTracker by lazy { UserAnalytics.getAnalyticsEventTracker() }
 
-    private fun setReturnAssistantProperty() {
+    private fun setReturnReasonsEventProperty() {
         userAnalyticsEventTracker.setUserProperty(
             setOf(
-                UserAnalyticsUserProperty.ReturnAssistantEnabled(
-                    GiniBank.getCaptureConfiguration()?.returnAssistantEnabled ?: false
-                ),
                 UserAnalyticsUserProperty.ReturnReasonsEnabled(GiniBank.enableReturnReasons),
             )
         )
@@ -83,7 +80,7 @@ class CaptureFlowFragment(private val openWithDocument: Document? = null) :
 
     override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
         super.onViewCreated(view, savedInstanceState)
-        setReturnAssistantProperty()
+        setReturnReasonsEventProperty()
         userAnalyticsEventTracker.trackEvent(UserAnalyticsEvent.SDK_OPENED)
         navController = (childFragmentManager.fragments[0]).findNavController()
     }

diff --git a/capture-sdk/sdk/src/main/java/net/gini/android/capture/GiniCaptureFragment.kt b/capture-sdk/sdk/src/main/java/net/gini/android/capture/GiniCaptureFragment.kt
@@ -27,6 +27,7 @@ import net.gini.android.capture.noresults.NoResultsFragment
 import net.gini.android.capture.review.multipage.MultiPageReviewFragment
 import net.gini.android.capture.tracking.useranalytics.UserAnalytics
 import net.gini.android.capture.tracking.useranalytics.properties.UserAnalyticsEventSuperProperty
+import net.gini.android.capture.tracking.useranalytics.properties.UserAnalyticsUserProperty
 import net.gini.android.capture.tracking.useranalytics.tracker.AmplitudeUserAnalyticsEventTracker
 import java.util.UUID
 
@@ -67,6 +68,10 @@ class GiniCaptureFragment(private val openWithDocument: Document? = null) :
             requireActivity().window.disallowScreenshots()
         }
 
+        setupUserAnalytics()
+    }
+
+    private fun setupUserAnalytics() {
         if (GiniCapture.hasInstance()) {
             UserAnalytics.initialize(requireActivity())
             val networkRequestsManager =
@@ -80,11 +85,24 @@ class GiniCaptureFragment(private val openWithDocument: Document? = null) :
                     ),
                     networkRequestsManager = networkRequestsManager
                 )
+                // set if return assistant is enabled for the client
+                res.configuration.isReturnAssistantEnabled.let {
+                    setReturnAssistantEventProperty(it)
+                }
             }
-
         }
     }
 
+    private fun setReturnAssistantEventProperty(isReturnAssistantEnabled: Boolean) {
+        userAnalyticsEventTracker.setUserProperty(
+            setOf(
+                UserAnalyticsUserProperty.ReturnAssistantEnabled(
+                    isReturnAssistantEnabled
+                )
+            )
+        )
+    }
+
     override fun onGetLayoutInflater(savedInstanceState: Bundle?): LayoutInflater {
         val inflater = super.onGetLayoutInflater(savedInstanceState)
         return this.getLayoutInflaterWithGiniCaptureTheme(inflater)
@@ -133,7 +151,7 @@ class GiniCaptureFragment(private val openWithDocument: Document? = null) :
         if (!didFinishWithResult && !willBeRestored) {
             giniCaptureFragmentListener.onFinishedWithResult(CaptureSDKResult.Cancel)
         }
-        if (willBeRestored)  {
+        if (willBeRestored) {
             UserAnalytics.flushEvents()
         }
     }