hotfix: fix password masking and otp length (mosip#52)

* fix: use otp length config * fix: fix password masking icon logic --------- Co-authored-by: Bunsy <[email protected]> Signed-off-by: Sreang Rathanak <[email protected]>
mengleang-0090 · Jan 15, 2024 · e342628 · e342628
1 parent 84ca7bd
commit e342628
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/signup-ui/src/components/ui/input.tsx b/signup-ui/src/components/ui/input.tsx
@@ -35,9 +35,9 @@ const Input = React.forwardRef<HTMLInputElement, InputProps>(
             onClick={togglePasswordVisibility}
           >
             {showPassword ? (
-              <Icons.eye className="h-5 w-5 text-gray-500" />
-            ) : (
               <Icons.eyeOff className="h-5 w-5 text-gray-500" />
+            ) : (
+              <Icons.eye className="h-5 w-5 text-gray-500" />
             )}
           </button>
         )}

diff --git a/signup-ui/src/pages/SignUpPage/SignUpPage.tsx b/signup-ui/src/pages/SignUpPage/SignUpPage.tsx
@@ -69,7 +69,11 @@ export const SignUpPage = ({ settings }: SignUpPageProps) => {
       }),
       // Step 2 - OTP Validation
       yup.object({
-        otp: yup.string().matches(/^\d{6}$/gm),
+        otp: yup
+          .string()
+          .matches(
+            new RegExp(`^\\d{${settings.response.configs["otp.length"]}}$`)
+          ),
       }),
       // Step 3 - Status Validation
       yup.object({}),