From 41b4d98b0ca6ce218d448b86588eb687f9f572eb Mon Sep 17 00:00:00 2001
From: Pritish Budhiraja <1805317@kiit.ac.in>
Date: Wed, 17 Apr 2024 12:05:54 +0530
Subject: [PATCH 1/2] =?UTF-8?q?Revert=20"feat:=20mandate=20Changes=20for?=
 =?UTF-8?q?=20the=20Saved=20card=20screen=20&=20SDK=20Button=20Loader=20c?=
 =?UTF-8?q?=E2=80=A6"?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This reverts commit 0e5b5abd588732481bfbafb84080b3297bf993c5.
---
 ...tailsCheckbox.res => AnimatedCheckbox.res} | 20 ++----
 src/Components/SavedMethods.res               | 40 ++----------
 src/Hooks/UtilityHooks.res                    | 13 ----
 src/Payments/CardPayment.res                  | 31 ++++-----
 src/Payments/PaymentMethodsRecord.res         |  3 -
 src/Utilities/PaymentBody.res                 | 65 ++++++-------------
 src/orca-loader/Hyper.res                     |  2 +-
 7 files changed, 47 insertions(+), 127 deletions(-)
 rename src/Components/{SaveDetailsCheckbox.res => AnimatedCheckbox.res} (68%)

diff --git a/src/Components/SaveDetailsCheckbox.res b/src/Components/AnimatedCheckbox.res
similarity index 68%
rename from src/Components/SaveDetailsCheckbox.res
rename to src/Components/AnimatedCheckbox.res
index 703c58858..7a6fd1726 100644
--- a/src/Components/SaveDetailsCheckbox.res
+++ b/src/Components/AnimatedCheckbox.res
@@ -1,14 +1,12 @@
 @react.component
-let make = (~isChecked, ~setIsChecked, ~list: PaymentMethodsRecord.list) => {
+let make = (~isChecked, ~setIsChecked) => {
   let {themeObj} = Recoil.useRecoilValueFromAtom(RecoilAtoms.configAtom)
-  let showFields = Recoil.useRecoilValueFromAtom(RecoilAtoms.showCardFieldsAtom)
-  let {business} = Recoil.useRecoilValueFromAtom(RecoilAtoms.optionAtom)
-
   let css = `.container {
   display: flex;
   cursor: pointer;
   position: relative;
   justify-content: center;
+  align-items: center;
 }
 
 .container input {
@@ -48,21 +46,13 @@ let make = (~isChecked, ~setIsChecked, ~list: PaymentMethodsRecord.list) => {
     ? ("Checkbox--checked", "CheckboxInput--checked", "CheckboxLabel--checked")
     : ("", "", "")
 
-  let valueToBeRendered = if showFields {
-    localeString.saveCardDetails
-  } else if business.name !== "" {
-    localeString.cardTerms(business.name)
-  } else {
-    localeString.cardTerms(list.merchant_name)
-  }
-
   <div className={`Checkbox ${checkboxState} flex flex-row gap-2 items-center`}>
     <style> {React.string(css)} </style>
     <label className={`container CheckboxInput ${checkedState}`}>
       <input type_={`checkbox`} onChange />
-      <div className={`checkmark CheckboxInput ${checkedState} mt-1`} />
-      <div className={`CheckboxLabel ${checkBoxLabelSate} ml-2 w-11/12`}>
-        {React.string(valueToBeRendered)}
+      <div className={`checkmark CheckboxInput ${checkedState}`} />
+      <div className={`CheckboxLabel ${checkBoxLabelSate} ml-2`}>
+        {React.string(localeString.saveCardDetails)}
       </div>
     </label>
   </div>
diff --git a/src/Components/SavedMethods.res b/src/Components/SavedMethods.res
index 3c7ee5724..944506758 100644
--- a/src/Components/SavedMethods.res
+++ b/src/Components/SavedMethods.res
@@ -1,3 +1,5 @@
+open CardUtils
+open Utils
 @react.component
 let make = (
   ~paymentToken,
@@ -6,12 +8,8 @@ let make = (
   ~loadSavedCards: PaymentType.savedCardsLoadState,
   ~cvcProps,
   ~paymentType,
-  ~list: PaymentMethodsRecord.list,
+  ~list,
 ) => {
-  open CardUtils
-  open Utils
-  open UtilityHooks
-
   let {themeObj, localeString} = Recoil.useRecoilValueFromAtom(RecoilAtoms.configAtom)
   let (showFields, setShowFields) = Recoil.useRecoilState(RecoilAtoms.showCardFieldsAtom)
   let areRequiredFieldsValid = Recoil.useRecoilValueFromAtom(RecoilAtoms.areRequiredFieldsValid)
@@ -21,9 +19,6 @@ let make = (
     postFailedSubmitResponse(~errortype="validation_error", ~message)
     loggerState.setLogError(~value=message, ~eventName=INVALID_FORMAT, ())
   }
-  let (isSaveCardsChecked, setIsSaveCardsChecked) = React.useState(_ => false)
-  let {displaySavedPaymentMethodsCheckbox} = Recoil.useRecoilValueFromAtom(RecoilAtoms.optionAtom)
-  let isGuestCustomer = useIsGuestCustomer()
 
   let intent = PaymentHelpers.usePaymentIntent(Some(loggerState), Card)
   let (token, _) = paymentToken
@@ -38,13 +33,6 @@ let make = (
     }
   }
 
-  let isCustomerAcceptanceRequired = useIsCustomerAcceptanceRequired(
-    ~displaySavedPaymentMethodsCheckbox,
-    ~isSaveCardsChecked,
-    ~list,
-    ~isGuestCustomer,
-  )
-
   let bottomElement = {
     savedMethods
     ->Array.mapWithIndex((obj, i) => {
@@ -102,7 +90,7 @@ let make = (
 
   let paymentType = customerMethod.paymentMethodType->Option.getOr(customerMethod.paymentMethod)
 
-  useHandlePostMessages(~complete, ~empty, ~paymentType, ~savedMethod=true)
+  UtilityHooks.useHandlePostMessages(~complete, ~empty, ~paymentType, ~savedMethod=true)
 
   let submitCallback = React.useCallback((ev: Window.event) => {
     let json = ev.data->JSON.parseExn
@@ -115,7 +103,6 @@ let make = (
         ~customerId,
         ~cvcNumber,
         ~requiresCvv=customerMethod.requiresCvv,
-        ~isCustomerAcceptanceRequired,
       )
     | _ => {
         let paymentMethodType = switch customerMethod.paymentMethodType {
@@ -128,7 +115,6 @@ let make = (
           ~customerId,
           ~paymentMethod=customerMethod.paymentMethod,
           ~paymentMethodType,
-          ~isCustomerAcceptanceRequired,
         )
       }
     }
@@ -167,20 +153,9 @@ let make = (
         }
       }
     }
-  }, (
-    areRequiredFieldsValid,
-    requiredFieldsBody,
-    empty,
-    complete,
-    customerMethod,
-    isCustomerAcceptanceRequired,
-  ))
+  }, (areRequiredFieldsValid, requiredFieldsBody, empty, complete, customerMethod))
   useSubmitPaymentData(submitCallback)
 
-  let conditionsForShowingSaveCardCheckbox = React.useMemo(() => {
-    !isGuestCustomer && list.payment_type !== SETUP_MANDATE && displaySavedPaymentMethodsCheckbox
-  }, (isGuestCustomer, list.payment_type, displaySavedPaymentMethodsCheckbox))
-
   <div className="flex flex-col overflow-auto h-auto no-scrollbar animate-slowShow">
     {if savedCardlength === 0 && (loadSavedCards === PaymentType.LoadingSavedCards || !showFields) {
       <div
@@ -204,11 +179,6 @@ let make = (
     } else {
       <RenderIf condition={!showFields}> {bottomElement} </RenderIf>
     }}
-    <RenderIf condition={conditionsForShowingSaveCardCheckbox}>
-      <div className="pt-4 pb-2 flex items-center justify-start">
-        <SaveDetailsCheckbox isChecked=isSaveCardsChecked setIsChecked=setIsSaveCardsChecked list />
-      </div>
-    </RenderIf>
     <RenderIf condition={!showFields}>
       <div
         className="Label flex flex-row gap-3 items-end cursor-pointer"
diff --git a/src/Hooks/UtilityHooks.res b/src/Hooks/UtilityHooks.res
index 02380e144..f34078b30 100644
--- a/src/Hooks/UtilityHooks.res
+++ b/src/Hooks/UtilityHooks.res
@@ -21,16 +21,3 @@ let useHandlePostMessages = (~complete, ~empty, ~paymentType, ~savedMethod=false
     None
   }, (complete, empty, paymentType))
 }
-
-let useIsCustomerAcceptanceRequired = (
-  ~displaySavedPaymentMethodsCheckbox,
-  ~isSaveCardsChecked,
-  ~list: PaymentMethodsRecord.list,
-  ~isGuestCustomer,
-) => React.useMemo(() => {
-    if displaySavedPaymentMethodsCheckbox {
-      isSaveCardsChecked || list.payment_type === SETUP_MANDATE
-    } else {
-      !(isGuestCustomer || list.payment_type === NORMAL)
-    }
-  }, (isSaveCardsChecked, list.payment_type))
diff --git a/src/Payments/CardPayment.res b/src/Payments/CardPayment.res
index 790ff4e5e..528aaee35 100644
--- a/src/Payments/CardPayment.res
+++ b/src/Payments/CardPayment.res
@@ -1,5 +1,7 @@
 type target = {checked: bool}
 type event = {target: target}
+open PaymentType
+open PaymentModeType
 
 @react.component
 let make = (
@@ -10,11 +12,7 @@ let make = (
   ~paymentType: CardThemeType.mode,
   ~list: PaymentMethodsRecord.list,
 ) => {
-  open PaymentType
-  open PaymentModeType
   open Utils
-  open UtilityHooks
-
   let {config, themeObj, localeString} = Recoil.useRecoilValueFromAtom(RecoilAtoms.configAtom)
   let options = Recoil.useRecoilValueFromAtom(RecoilAtoms.optionAtom)
   let loggerState = Recoil.useRecoilValueFromAtom(RecoilAtoms.loggerAtom)
@@ -83,9 +81,13 @@ let make = (
     None
   }, [complete])
 
-  useHandlePostMessages(~complete=complete && areRequiredFieldsValid, ~empty, ~paymentType="card")
+  UtilityHooks.useHandlePostMessages(
+    ~complete=complete && areRequiredFieldsValid,
+    ~empty,
+    ~paymentType="card",
+  )
 
-  let isGuestCustomer = useIsGuestCustomer()
+  let isGuestCustomer = UtilityHooks.useIsGuestCustomer()
   let isCvcValidValue = CardUtils.getBoolOptionVal(isCVCValid)
   let (cardEmpty, cardComplete, cardInvalid) = CardUtils.useCardDetails(
     ~cvcNumber,
@@ -93,12 +95,13 @@ let make = (
     ~isCvcValidValue,
   )
 
-  let isCustomerAcceptanceRequired = useIsCustomerAcceptanceRequired(
-    ~displaySavedPaymentMethodsCheckbox,
-    ~isSaveCardsChecked,
-    ~list,
-    ~isGuestCustomer,
-  )
+  let isCustomerAcceptanceRequired = React.useMemo(() => {
+    if displaySavedPaymentMethodsCheckbox {
+      isSaveCardsChecked || list.payment_type === SETUP_MANDATE
+    } else {
+      !(isGuestCustomer || list.payment_type === NORMAL)
+    }
+  }, (isSaveCardsChecked, list.payment_type))
 
   let submitCallback = React.useCallback((ev: Window.event) => {
     let json = ev.data->JSON.parseExn
@@ -273,9 +276,7 @@ let make = (
           />
           <RenderIf condition={conditionsForShowingSaveCardCheckbox}>
             <div className="pt-4 pb-2 flex items-center justify-start">
-              <SaveDetailsCheckbox
-                isChecked=isSaveCardsChecked setIsChecked=setIsSaveCardsChecked list
-              />
+              <AnimatedCheckbox isChecked=isSaveCardsChecked setIsChecked=setIsSaveCardsChecked />
             </div>
           </RenderIf>
           <RenderIf condition={isCustomerAcceptanceRequired}>
diff --git a/src/Payments/PaymentMethodsRecord.res b/src/Payments/PaymentMethodsRecord.res
index 636541c8b..19b8774a2 100644
--- a/src/Payments/PaymentMethodsRecord.res
+++ b/src/Payments/PaymentMethodsRecord.res
@@ -724,7 +724,6 @@ type list = {
   payment_methods: array<methods>,
   mandate_payment: option<mandate>,
   payment_type: payment_type,
-  merchant_name: string,
 }
 
 open Utils
@@ -746,7 +745,6 @@ let defaultList = {
   payment_methods: [],
   mandate_payment: None,
   payment_type: NONE,
-  merchant_name: "",
 }
 let getMethod = str => {
   switch str {
@@ -965,7 +963,6 @@ let itemToObjMapper = dict => {
     payment_methods: getMethodsArr(dict, "payment_methods"),
     mandate_payment: getMandate(dict, "mandate_payment"),
     payment_type: getString(dict, "payment_type", "")->paymentTypeMapper,
-    merchant_name: getString(dict, "merchant_name", ""),
   }
 }
 
diff --git a/src/Utilities/PaymentBody.res b/src/Utilities/PaymentBody.res
index 9ddae8ef2..740d8c93f 100644
--- a/src/Utilities/PaymentBody.res
+++ b/src/Utilities/PaymentBody.res
@@ -64,6 +64,20 @@ let boletoBody = (~socialSecurityNumber) => [
   ),
 ]
 
+let savedCardBody = (~paymentToken, ~customerId, ~cvcNumber, ~requiresCvv) => {
+  let savedCardBody = [
+    ("payment_method", "card"->JSON.Encode.string),
+    ("payment_token", paymentToken->JSON.Encode.string),
+    ("customer_id", customerId->JSON.Encode.string),
+  ]
+
+  if requiresCvv {
+    savedCardBody->Array.push(("card_cvc", cvcNumber->JSON.Encode.string))->ignore
+  }
+
+  savedCardBody
+}
+
 let customerAcceptanceBody =
   [
     ("acceptance_type", "online"->JSON.Encode.string),
@@ -78,51 +92,12 @@ let customerAcceptanceBody =
   ->Dict.fromArray
   ->JSON.Encode.object
 
-let savedPaymentMethodBody = (
-  ~paymentToken,
-  ~customerId,
-  ~paymentMethod,
-  ~paymentMethodType,
-  ~isCustomerAcceptanceRequired,
-) => {
-  let savedPaymentMethodBody = [
-    ("payment_method", paymentMethod->JSON.Encode.string),
-    ("payment_token", paymentToken->JSON.Encode.string),
-    ("customer_id", customerId->JSON.Encode.string),
-    ("payment_method_type", paymentMethodType),
-  ]
-
-  if isCustomerAcceptanceRequired {
-    savedPaymentMethodBody->Array.push(("customer_acceptance", customerAcceptanceBody))->ignore
-  }
-
-  savedPaymentMethodBody
-}
-
-let savedCardBody = (
-  ~paymentToken,
-  ~customerId,
-  ~cvcNumber,
-  ~requiresCvv,
-  ~isCustomerAcceptanceRequired,
-) => {
-  let savedCardBody = [
-    ("payment_method", "card"->JSON.Encode.string),
-    ("payment_token", paymentToken->JSON.Encode.string),
-    ("customer_id", customerId->JSON.Encode.string),
-    ("customer_acceptance", customerAcceptanceBody),
-  ]
-
-  if requiresCvv {
-    savedCardBody->Array.push(("card_cvc", cvcNumber->JSON.Encode.string))->ignore
-  }
-
-  if isCustomerAcceptanceRequired {
-    savedCardBody->Array.push(("customer_acceptance", customerAcceptanceBody))->ignore
-  }
-
-  savedCardBody
-}
+let savedPaymentMethodBody = (~paymentToken, ~customerId, ~paymentMethod, ~paymentMethodType) => [
+  ("payment_method", paymentMethod->JSON.Encode.string),
+  ("payment_token", paymentToken->JSON.Encode.string),
+  ("customer_id", customerId->JSON.Encode.string),
+  ("payment_method_type", paymentMethodType),
+]
 
 let mandateBody = paymentType => {
   [
diff --git a/src/orca-loader/Hyper.res b/src/orca-loader/Hyper.res
index 1a8a955b8..36dc9ae10 100644
--- a/src/orca-loader/Hyper.res
+++ b/src/orca-loader/Hyper.res
@@ -318,7 +318,7 @@ let make = (publishableKey, options: option<JSON.t>, analyticsInfo: option<JSON.
                     },
                   )
                 }
-                postSubmitMessage(dict)
+
                 if val->JSON.Decode.bool->Option.getOr(false) && redirect === "always" {
                   Window.replace(returnUrl)
                 } else if !(val->JSON.Decode.bool->Option.getOr(false)) {

From 6643d6955f4fc7fb94c1a6a1c890821cad44f85d Mon Sep 17 00:00:00 2001
From: Pritish Budhiraja <1805317@kiit.ac.in>
Date: Wed, 17 Apr 2024 12:23:03 +0530
Subject: [PATCH 2/2] fix: mandate changes

---
 src/Payments/CardPayment.res          | 14 +++++++++-----
 src/Payments/PaymentMethodsRecord.res |  3 +++
 src/orca-loader/Hyper.res             |  2 +-
 3 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/src/Payments/CardPayment.res b/src/Payments/CardPayment.res
index 528aaee35..5757038dd 100644
--- a/src/Payments/CardPayment.res
+++ b/src/Payments/CardPayment.res
@@ -190,6 +190,8 @@ let make = (
 
   let nicknameFieldClassName = conditionsForShowingSaveCardCheckbox ? "pt-2" : "pt-5"
 
+  Js.log2("sdvjnsddsjd", (list.mandate_payment, options.terms.card))
+
   <div className="animate-slowShow">
     <RenderIf condition={showFields || isBancontact}>
       <div
@@ -293,19 +295,21 @@ let make = (
       />
     </RenderIf>
     <RenderIf condition={!isBancontact}>
-      {switch (list.mandate_payment, options.terms.card) {
-      | (Some(_), Auto)
-      | (_, Always) =>
+      {switch (list.mandate_payment, options.terms.card, list.payment_type) {
+      | (Some(_), Auto, _)
+      | (_, Always, _)
+      | (_, _, NEW_MANDATE)
+      | (_, _, SETUP_MANDATE) =>
         <div
           className="opacity-50 text-xs mb-2 text-left"
           style={ReactDOMStyle.make(
             ~color=themeObj.colorText,
-            ~marginTop=themeObj.spacingGridColumn->addSize(10.0, Pixel),
+            ~marginTop=themeObj.spacingGridColumn,
             (),
           )}>
           <Terms mode={Card} />
         </div>
-      | (_, _) => React.null
+      | (_, _, _) => React.null
       }}
     </RenderIf>
   </div>
diff --git a/src/Payments/PaymentMethodsRecord.res b/src/Payments/PaymentMethodsRecord.res
index 19b8774a2..636541c8b 100644
--- a/src/Payments/PaymentMethodsRecord.res
+++ b/src/Payments/PaymentMethodsRecord.res
@@ -724,6 +724,7 @@ type list = {
   payment_methods: array<methods>,
   mandate_payment: option<mandate>,
   payment_type: payment_type,
+  merchant_name: string,
 }
 
 open Utils
@@ -745,6 +746,7 @@ let defaultList = {
   payment_methods: [],
   mandate_payment: None,
   payment_type: NONE,
+  merchant_name: "",
 }
 let getMethod = str => {
   switch str {
@@ -963,6 +965,7 @@ let itemToObjMapper = dict => {
     payment_methods: getMethodsArr(dict, "payment_methods"),
     mandate_payment: getMandate(dict, "mandate_payment"),
     payment_type: getString(dict, "payment_type", "")->paymentTypeMapper,
+    merchant_name: getString(dict, "merchant_name", ""),
   }
 }
 
diff --git a/src/orca-loader/Hyper.res b/src/orca-loader/Hyper.res
index 36dc9ae10..1a8a955b8 100644
--- a/src/orca-loader/Hyper.res
+++ b/src/orca-loader/Hyper.res
@@ -318,7 +318,7 @@ let make = (publishableKey, options: option<JSON.t>, analyticsInfo: option<JSON.
                     },
                   )
                 }
-
+                postSubmitMessage(dict)
                 if val->JSON.Decode.bool->Option.getOr(false) && redirect === "always" {
                   Window.replace(returnUrl)
                 } else if !(val->JSON.Decode.bool->Option.getOr(false)) {