From 25dcfecfaf2070ab71a8dd0e88d8fa20f335a6c9 Mon Sep 17 00:00:00 2001 From: victorAnumudu Date: Thu, 27 Apr 2023 10:47:05 +0100 Subject: [PATCH 1/2] made validates that account number matches retype account number --- src/components/MyWallet/AddRecipient.jsx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/components/MyWallet/AddRecipient.jsx b/src/components/MyWallet/AddRecipient.jsx index f8d778b..b54c828 100644 --- a/src/components/MyWallet/AddRecipient.jsx +++ b/src/components/MyWallet/AddRecipient.jsx @@ -30,9 +30,8 @@ const validationSchema = Yup.object().shape({ .max(25, 'Maximum 25 characters') .required('Account Number is required'), repeatAccountNumber: Yup.string() - .min(3, 'Minimum 3 characters') - .max(25, 'Maximum 25 characters') - .required('Repeat Password is required'), + .required('Repeat Account Number is required') + .oneOf([Yup.ref('accountNumber'), null], 'Must match Account Number'), accountType: Yup.string() .min(3, 'Minimum 3 characters') .max(25, 'Maximum 25 characters') From ecdd8f9180bb87d0fec901c6938b4cf191273797 Mon Sep 17 00:00:00 2001 From: victorAnumudu Date: Thu, 27 Apr 2023 11:08:11 +0100 Subject: [PATCH 2/2] validates that account number is a number --- src/components/MyWallet/AddRecipient.jsx | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/components/MyWallet/AddRecipient.jsx b/src/components/MyWallet/AddRecipient.jsx index b54c828..9d3752d 100644 --- a/src/components/MyWallet/AddRecipient.jsx +++ b/src/components/MyWallet/AddRecipient.jsx @@ -26,9 +26,10 @@ const validationSchema = Yup.object().shape({ .max(25, 'Maximum 25 characters') .required('Bank name is required'), accountNumber: Yup.string() - .min(3, 'Minimum 3 characters') - .max(25, 'Maximum 25 characters') - .required('Account Number is required'), + .matches(/\d/, "must be a number") + .min(3, 'Minimum 3 characters') + .max(25, 'Maximum 25 characters') + .required('Account Number is required'), repeatAccountNumber: Yup.string() .required('Repeat Account Number is required') .oneOf([Yup.ref('accountNumber'), null], 'Must match Account Number'),