fix: HS-154: Log redirects in case of confirm error, remove extra logs (

#6)
juspay · Nov 6, 2023 · 2f57f01 · 2f57f01
1 parent b084b4e
commit 2f57f01
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 7 deletions.
diff --git a/src/Utilities/LoggerUtils.res b/src/Utilities/LoggerUtils.res
@@ -75,10 +75,12 @@ let handleLogging = (
   ~internalMetadata="",
   ~eventName,
   ~paymentMethod,
+  ~logType: OrcaLogger.logType=INFO,
   (),
 ) => {
   switch optLogger {
-  | Some(logger) => logger.setLogInfo(~value, ~internalMetadata, ~eventName, ~paymentMethod, ())
+  | Some(logger) =>
+    logger.setLogInfo(~value, ~internalMetadata, ~eventName, ~paymentMethod, ~logType, ())
   | _ => ()
   }
 }
diff --git a/src/Utilities/PaymentHelpers.res b/src/Utilities/PaymentHelpers.res
@@ -302,6 +302,15 @@ let intentCall = (
               ~message="Payment failed. Try again!",
             )
             if uri->Js.String2.includes("force_sync=true") {
+              handleLogging(
+                ~optLogger,
+                ~value=intent.nextAction.type_,
+                ~internalMetadata=intent.nextAction.type_,
+                ~eventName=REDIRECTING_USER,
+                ~paymentMethod,
+                ~logType=ERROR,
+                (),
+              )
               openUrl(url.href)
             }
           }

diff --git a/src/orca-loader/Elements.res b/src/orca-loader/Elements.res
@@ -536,12 +536,6 @@ let make = (
                                         ("applePaySyncPayment", true->Js.Json.boolean),
                                       ]->Js.Dict.fromArray
                                     mountedIframeRef->Window.iframePostMessage(msg)
-                                    logger.setLogInfo(
-                                      ~value="",
-                                      ~eventName=PAYMENT_DATA_FILLED,
-                                      ~paymentMethod="APPLE_PAY",
-                                      (),
-                                    )
                                     resolve()
                                   })
                                   ->catch(err => {