From 8d9c6202fd479c24a01673997bd8d8ab372cf496 Mon Sep 17 00:00:00 2001
From: Pritish Budhiraja <pritish.budhiraja@gmail.com>
Date: Tue, 9 Jul 2024 16:32:43 +0530
Subject: [PATCH] fix: date of birth validations (#480)

Co-authored-by: vsrivatsa-juspay <vrishab.srivatsa@juspay.in>
---
 src/Payments/DateOfBirth.res         | 45 +++++++++++++++++++++++++++-
 src/Utilities/DynamicFieldsUtils.res | 22 ++++++++++++--
 src/Utilities/PaymentHelpers.res     |  2 +-
 src/Utilities/RecoilAtoms.res        |  2 +-
 src/Utilities/Utils.res              |  9 ++++++
 src/libraries/DatePicker.res         |  5 ++--
 src/orca-log-catcher/OrcaLogger.res  | 15 ++--------
 7 files changed, 80 insertions(+), 20 deletions(-)

diff --git a/src/Payments/DateOfBirth.res b/src/Payments/DateOfBirth.res
index c1637f260..8992a1abe 100644
--- a/src/Payments/DateOfBirth.res
+++ b/src/Payments/DateOfBirth.res
@@ -21,8 +21,37 @@ let years = Array.fromInitializer(~length=currentYear - startYear, i => currentY
 
 @react.component
 let make = () => {
+  open Utils
   let {themeObj, localeString} = Recoil.useRecoilValueFromAtom(RecoilAtoms.configAtom)
   let (selectedDate, setSelectedDate) = Recoil.useRecoilState(RecoilAtoms.dateOfBirth)
+  let (error, setError) = React.useState(_ => false)
+  let isNotEligible = React.useMemo(() => {
+    let isAbove18 = switch selectedDate->Nullable.toOption {
+    | Some(val) => val->checkIs18OrAbove
+    | None => false
+    }
+    !isAbove18
+  }, [selectedDate])
+
+  let submitCallback = React.useCallback((ev: Window.event) => {
+    let json = ev.data->JSON.parseExn
+    let confirm = json->getDictFromJson->ConfirmType.itemToObjMapper
+    if confirm.doSubmit {
+      switch selectedDate->Nullable.toOption {
+      | Some(_) => setError(_ => false)
+      | None => ()
+      }
+    }
+  }, (selectedDate, isNotEligible))
+
+  useSubmitPaymentData(submitCallback)
+
+  let onChange = date => {
+    setSelectedDate(_ => date)
+  }
+  let errorString = error
+    ? "Date of birth is required"
+    : "Age should be equal or greater than 18 years"
 
   <div className="flex flex-col gap-1">
     <div
@@ -39,9 +68,11 @@ let make = () => {
       icon={<Icon name="calander" size=13 className="!px-[6px] !py-[10px]" />}
       className="w-full border border-gray-300 rounded p-2"
       selected={selectedDate}
-      onChange={date => setSelectedDate(_ => date)}
+      onChange={date => onChange(date)}
       dateFormat="dd-MM-yyyy"
       wrapperClassName="datepicker"
+      shouldCloseOnSelect=true
+      placeholderText="Enter Date of Birth"
       renderCustomHeader={val => {
         <div className="flex gap-4 items-center justify-center m-2">
           <select
@@ -75,5 +106,17 @@ let make = () => {
         </div>
       }}
     />
+    <RenderIf condition={error || isNotEligible}>
+      <div
+        className="Error pt-1"
+        style={
+          color: themeObj.colorDangerText,
+          fontSize: themeObj.fontSizeSm,
+          alignSelf: "start",
+          textAlign: "left",
+        }>
+        {React.string(errorString)}
+      </div>
+    </RenderIf>
   </div>
 }
diff --git a/src/Utilities/DynamicFieldsUtils.res b/src/Utilities/DynamicFieldsUtils.res
index 52650ceb9..ffeb3a9d6 100644
--- a/src/Utilities/DynamicFieldsUtils.res
+++ b/src/Utilities/DynamicFieldsUtils.res
@@ -126,6 +126,7 @@ let useRequiredFieldsEmptyAndValid = (
   let setAreRequiredFieldsEmpty = Recoil.useSetRecoilState(areRequiredFieldsEmpty)
   let {billingAddress} = Recoil.useRecoilValueFromAtom(optionAtom)
   let cryptoCurrencyNetworks = Recoil.useRecoilValueFromAtom(cryptoCurrencyNetworks)
+  let dateOfBirth = Recoil.useRecoilValueFromAtom(dateOfBirth)
 
   let fieldsArrWithBillingAddress = fieldsArr->addBillingAddressIfUseBillingAddress(billingAddress)
 
@@ -159,6 +160,11 @@ let useRequiredFieldsEmptyAndValid = (
         isExpiryValid->Option.getOr(false)
       | CardCvc => isCVCValid->Option.getOr(false)
       | CardExpiryAndCvc => isExpiryValid->Option.getOr(false) && isCVCValid->Option.getOr(false)
+      | DateOfBirth =>
+        switch dateOfBirth->Nullable.toOption {
+        | Some(val) => val->Utils.checkIs18OrAbove
+        | None => false
+        }
       | _ => true
       }
     })
@@ -203,6 +209,7 @@ let useRequiredFieldsEmptyAndValid = (
       | CardExpiryAndCvc =>
         let (month, year) = getExpiryDates(cardExpiry)
         month === "" || year === "" || cvcNumber === ""
+      | DateOfBirth => dateOfBirth->Js.Nullable.isNullable
       | _ => false
       }
     })
@@ -215,6 +222,7 @@ let useRequiredFieldsEmptyAndValid = (
     country,
     billingName.value,
     line1.value,
+    dateOfBirth,
     (
       email,
       line2.value,
@@ -382,8 +390,12 @@ let useSetInitialRequiredFields = (
           setCryptoCurrencyNetworks(_ => value)
         }
       | DateOfBirth =>
-        if value !== "" && dateOfBirth->Date.toDateString === "" {
-          setDateOfBirth(_ => dateOfBirth)
+        switch dateOfBirth->Nullable.toOption {
+        | Some(x) =>
+          if value !== "" && x->Date.toDateString === "" {
+            setDateOfBirth(_ => Nullable.make(x))
+          }
+        | None => ()
         }
       | SpecialField(_)
       | InfoElement
@@ -470,7 +482,11 @@ let useRequiredFieldsBody = (
       let (_, year) = CardUtils.getExpiryDates(cardExpiry)
       year
     | CryptoCurrencyNetworks => cryptoCurrencyNetworks
-    | DateOfBirth => dateOfBirth->Date.toISOString->String.slice(~start=0, ~end=10)
+    | DateOfBirth =>
+      switch dateOfBirth->Nullable.toOption {
+      | Some(x) => x->Date.toISOString->String.slice(~start=0, ~end=10)
+      | None => ""
+      }
     | CardCvc => cvcNumber
     | StateAndCity
     | CountryAndPincode(_)
diff --git a/src/Utilities/PaymentHelpers.res b/src/Utilities/PaymentHelpers.res
index b27ee91c5..d6b5591b9 100644
--- a/src/Utilities/PaymentHelpers.res
+++ b/src/Utilities/PaymentHelpers.res
@@ -227,7 +227,7 @@ let retrieveStatus = (~headers, ~switchToCustomPod, pollID, logger) => {
     ~optLogger=Some(logger),
     ~url=uri,
     ~apiLogType=Request,
-    ~eventName=POLL_STATUS_INIT,
+    ~eventName=POLL_STATUS_CALL_INIT,
     ~logType=INFO,
     ~logCategory=API,
     (),
diff --git a/src/Utilities/RecoilAtoms.res b/src/Utilities/RecoilAtoms.res
index 0d9e2e359..1e3dd1c7a 100644
--- a/src/Utilities/RecoilAtoms.res
+++ b/src/Utilities/RecoilAtoms.res
@@ -61,7 +61,7 @@ let cryptoCurrencyNetworks = Recoil.atom("cryptoCurrencyNetworks", "")
 let isShowOrPayUsing = Recoil.atom("isShowOrPayUsing", false)
 let areRequiredFieldsValid = Recoil.atom("areRequiredFieldsValid", true)
 let areRequiredFieldsEmpty = Recoil.atom("areRequiredFieldsEmpty", false)
-let dateOfBirth = Recoil.atom("dateOfBirth", Date.make())
+let dateOfBirth = Recoil.atom("dateOfBirth", Nullable.null)
 let userBillingName = Recoil.atom("userBillingName", defaultFieldValues)
 
 type areOneClickWalletsRendered = {
diff --git a/src/Utilities/Utils.res b/src/Utilities/Utils.res
index 663020fa2..ecaafd04f 100644
--- a/src/Utilities/Utils.res
+++ b/src/Utilities/Utils.res
@@ -1341,3 +1341,12 @@ let handleFailureResponse = (~message, ~errorType) =>
 
 let getPaymentId = clientSecret =>
   String.split(clientSecret, "_secret_")->Array.get(0)->Option.getOr("")
+
+let checkIs18OrAbove = dateOfBirth => {
+  let currentDate = Date.make()
+  let year = currentDate->Date.getFullYear - 18
+  let month = currentDate->Date.getMonth
+  let date = currentDate->Date.getDate
+  let compareDate = Date.makeWithYMD(~year, ~month, ~date)
+  dateOfBirth <= compareDate
+}
diff --git a/src/libraries/DatePicker.res b/src/libraries/DatePicker.res
index a6bd909ee..13a3205cc 100644
--- a/src/libraries/DatePicker.res
+++ b/src/libraries/DatePicker.res
@@ -18,8 +18,8 @@ type customHeaderProps = {
 
 @module("react-datepicker") @react.component
 external make: (
-  ~selected: Date.t,
-  ~onChange: Date.t => unit,
+  ~selected: Nullable.t<Date.t>,
+  ~onChange: Nullable.t<Date.t> => unit,
   ~showIcon: bool=?,
   ~icon: React.element=?,
   ~dateFormat: string=?,
@@ -31,4 +31,5 @@ external make: (
   ~className: string=?,
   ~wrapperClassName: string=?,
   ~closeOnScroll: bool=?,
+  ~shouldCloseOnSelect: bool=?,
 ) => React.element = "default"
diff --git a/src/orca-log-catcher/OrcaLogger.res b/src/orca-log-catcher/OrcaLogger.res
index 876b1672a..f673588ef 100644
--- a/src/orca-log-catcher/OrcaLogger.res
+++ b/src/orca-log-catcher/OrcaLogger.res
@@ -70,7 +70,7 @@ type eventName =
   | PAYMENT_METHODS_RESPONSE
   | LOADER_CHANGED
   | PAYMENT_SESSION_INITIATED
-  | POLL_STATUS_INIT
+  | POLL_STATUS_CALL_INIT
   | POLL_STATUS_CALL
   | COMPLETE_AUTHORIZE_CALL_INIT
   | COMPLETE_AUTHORIZE_CALL
@@ -151,7 +151,7 @@ let eventNameToStrMapper = eventName => {
   | PAYMENT_METHODS_RESPONSE => "PAYMENT_METHODS_RESPONSE"
   | LOADER_CHANGED => "LOADER_CHANGED"
   | PAYMENT_SESSION_INITIATED => "PAYMENT_SESSION_INITIATED"
-  | POLL_STATUS_INIT => "POLL_STATUS_INIT"
+  | POLL_STATUS_CALL_INIT => "POLL_STATUS_CALL_INIT"
   | POLL_STATUS_CALL => "POLL_STATUS_CALL"
   | COMPLETE_AUTHORIZE_CALL_INIT => "COMPLETE_AUTHORIZE_CALL_INIT"
   | COMPLETE_AUTHORIZE_CALL => "COMPLETE_AUTHORIZE_CALL"
@@ -649,15 +649,7 @@ let make = (
         | _ => 0.
         }
       }
-    | AUTHENTICATION_CALL
-    | RETRIEVE_CALL
-    | CONFIRM_CALL
-    | SESSIONS_CALL
-    | PAYMENT_METHODS_CALL
-    | CUSTOMER_PAYMENT_METHODS_CALL
-    | PAYMENT_METHODS_AUTH_EXCHANGE_CALL
-    | PAYMENT_METHODS_AUTH_LINK_CALL
-    | CREATE_CUSTOMER_PAYMENT_METHODS_CALL => {
+    | _ => {
         let logRequestTimestamp =
           events.contents->Dict.get(eventName->eventNameToStrMapper ++ "_INIT")
         switch (logRequestTimestamp, apiLogType) {
@@ -666,7 +658,6 @@ let make = (
         | _ => 0.
         }
       }
-    | _ => 0.
     }
     latency > 0. ? latency->Float.toString : ""
   }