From d9e1c91a6bf54627409a362bba817bfd7606d90c Mon Sep 17 00:00:00 2001 From: Vaibhav Surve Date: Tue, 8 Apr 2025 16:10:42 +0530 Subject: [PATCH 1/2] Refactor project status options and remove "Suspended" status --- src/components/Project/ManageProjectInfo.jsx | 6 +++--- src/components/Project/ProjectCard.jsx | 6 +++--- src/utils/projectStatus.js | 8 ++++---- 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/components/Project/ManageProjectInfo.jsx b/src/components/Project/ManageProjectInfo.jsx index 2c55ac54..96a51d7e 100644 --- a/src/components/Project/ManageProjectInfo.jsx +++ b/src/components/Project/ManageProjectInfo.jsx @@ -181,10 +181,10 @@ const ManageProjectInfo = ( {project,handleSubmitForm, onClose} ) => - - + {/* */} + - + {errors.projectStatusId &&
{errors.projectStatusId.message}
} diff --git a/src/components/Project/ProjectCard.jsx b/src/components/Project/ProjectCard.jsx index 4804d19e..39fae552 100644 --- a/src/components/Project/ProjectCard.jsx +++ b/src/components/Project/ProjectCard.jsx @@ -30,11 +30,11 @@ const ProjectCard = ( {projectData} ) => return "Active"; case 2: return "On Hold"; + // case 3: + // return "Suspended"; case 3: - return "Suspended"; - case 4: return "Inactive"; - case 5: + case 4: return "Completed"; } }; diff --git a/src/utils/projectStatus.js b/src/utils/projectStatus.js index c93ec488..306359e4 100644 --- a/src/utils/projectStatus.js +++ b/src/utils/projectStatus.js @@ -6,13 +6,13 @@ export const ProjectStatus =(statusId)=>{ case 2: return "On Hold" break; + // case 3: + // return "Suspended" + // break; case 3: - return "Suspended" - break; - case 4: return "Inactive" break; - case 5: + case 4: return "Completed" break; default: From 9d9569460a2196b16e10ea6d49388377c9435ed2 Mon Sep 17 00:00:00 2001 From: Vaibhav Surve Date: Tue, 8 Apr 2025 17:33:26 +0530 Subject: [PATCH 2/2] Enhance email validation and fix spelling errors in address fields --- src/components/Employee/ManageEmployee.jsx | 45 ++++++++++++++++------ src/pages/employee/EmployeeProfile.jsx | 2 +- 2 files changed, 35 insertions(+), 12 deletions(-) diff --git a/src/components/Employee/ManageEmployee.jsx b/src/components/Employee/ManageEmployee.jsx index 349b33cb..3b7fe363 100644 --- a/src/components/Employee/ManageEmployee.jsx +++ b/src/components/Employee/ManageEmployee.jsx @@ -36,7 +36,30 @@ const ManageEmployee = () => { FirstName: z.string().min(1, { message: "First Name is required" }), MiddleName: z.string().optional(), LastName: z.string().min(1, { message: "Last Name is required" }), - Email: z.string().optional(), + Email: z + .string() + .optional() + .refine( + (val) => + !val || /^[^\s@]+@[^\s@]+\.[^\s@]+$/.test(val), + { + message: "Invalid email format", + } + ) + .refine( + (val) => { + if (!val) return true; + const [local, domain] = val.split("@"); + return ( + val.length <= 320 && + local?.length <= 64 && + domain?.length <= 255 + ); + }, + { + message: "Email local or domain part is too long", + } + ), CurrentAddress: z .string() .min(1, { message: "Current Address is required" }) @@ -86,7 +109,7 @@ const ManageEmployee = () => { .refine((val) => !val || /^[A-Z]{5}[0-9]{4}[A-Z]{1}$/.test(val), { message: "Invalid PAN number", }), - PeramnentAddress: z + PermanentAddress: z .string() .min(1, { message: "Permanent Address is required" }) .max(150, { message: "Address cannot exceed 150 characters" }), @@ -120,7 +143,7 @@ const ManageEmployee = () => { AadharNumber: currentEmployee?.aadharNumber || "", Gender: currentEmployee?.gender || "", PanNumber: currentEmployee?.panNumber || "", - PeramnentAddress: currentEmployee?.peramnentAddress || "", + PermanentAddress: currentEmployee?.permanentAddress || "", PhoneNumber: currentEmployee?.phoneNumber || "", JobRoleId: currentEmployee?.jobRoleId || "", }, @@ -191,7 +214,7 @@ const ManageEmployee = () => { AadharNumber: currentEmployee.aadharNumber || "", Gender: currentEmployee.gender || "", PanNumber: currentEmployee.panNumber || "", - PeramnentAddress: currentEmployee.peramnentAddress || "", + PermanentAddress: currentEmployee.permanentAddress || "", PhoneNumber: currentEmployee.phoneNumber || "", JobRoleId: currentEmployee.jobRoleId?.toString() || "", } @@ -421,22 +444,22 @@ const ManageEmployee = () => { )}
-
Permnant Address
+
Permanent Address
- {errors.PeramnentAddress && ( + {errors.PermanentAddress && (
- {errors.PeramnentAddress.message} + {errors.PermanentAddress.message}
)}
diff --git a/src/pages/employee/EmployeeProfile.jsx b/src/pages/employee/EmployeeProfile.jsx index 6987c322..a000f72d 100644 --- a/src/pages/employee/EmployeeProfile.jsx +++ b/src/pages/employee/EmployeeProfile.jsx @@ -174,7 +174,7 @@ const EmployeeProfile = () => {
  • - {currentEmployee?.peramnentAddress} + {currentEmployee?.permanentAddress}