diff --git a/src/components/Project/AssignRole.jsx b/src/components/Project/AssignRole.jsx index be557dd4..46de8044 100644 --- a/src/components/Project/AssignRole.jsx +++ b/src/components/Project/AssignRole.jsx @@ -53,16 +53,21 @@ const AssignRoleModel = ({ assignData, onClose }) => { }, }); - const handleRoleChange = (event) => { - setSelectedRole(event.plannedTask.value); + const handleRoleChange = ( event ) => + { + reset() + setSelectedEmployees( [] ) + setSelectedRole(event.target.value); }; const filteredEmployees = selectedRole === "all" ? employees - : employees.filter((emp) => emp.JobRoleId.toString() === selectedRole); + : employees.filter((emp) => + String(emp.jobRoleId || "") === selectedRole + ); + - // not need currently for this fun const handleEmployeeSelection = (employeeId, field) => { setSelectedEmployees((prevSelected) => { let updatedSelection; @@ -97,8 +102,7 @@ const AssignRoleModel = ({ assignData, onClose }) => { showToast( "Task Successfully Assigend", "success" ); setSelectedEmployees([]) reset(); - closeModal() - + onClose(); } catch (error) { showToast("something wrong", "error"); } @@ -109,7 +113,18 @@ const AssignRoleModel = ({ assignData, onClose }) => { }, [dispatch]); return ( - <> +
+
+
+

@@ -161,16 +176,17 @@ const AssignRoleModel = ({ assignData, onClose }) => {

Employee
- + {( filteredEmployees?.length === 0 && employees ) &&
No employees found
} +
{selectedRole !== "" && (
-
- {filteredEmployees.map((emp) => { +
+ {filteredEmployees?.map((emp) => { const jobRole = jobRoleData?.find( - (role) => role.id === emp.jobRoleId + (role) => role?.id === emp?.jobRoleId ); return ( @@ -188,9 +204,9 @@ const AssignRoleModel = ({ assignData, onClose }) => { {...field} className="form-check-input mx-2" type="checkbox" - id={`employee-${emp.id}`} + id={`employee-${emp?.id}`} value={emp.id} - checked={field.value.includes(emp.id)} // Ensure the checkbox reflects the current form state + checked={field.value.includes(emp.id)} onChange={() => { handleEmployeeSelection(emp.id, field); }} @@ -326,15 +342,17 @@ const AssignRoleModel = ({ assignData, onClose }) => { className="btn btn-sm btn-label-secondary" data-bs-dismiss="modal" aria-label="Close" - onClick={closeModal} + onClick={onClose} > Cancel
-
- +
+
+
+
); }; export default AssignRoleModel;