From ea6d03ef754b7c213253ff1d534701234015707d Mon Sep 17 00:00:00 2001 From: pramod mahajan Date: Wed, 16 Jul 2025 18:15:46 +0530 Subject: [PATCH 01/12] fiex pagination in regularization issue --- src/components/Activities/Regularization.jsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/components/Activities/Regularization.jsx b/src/components/Activities/Regularization.jsx index a192f51b..db58ed58 100644 --- a/src/components/Activities/Regularization.jsx +++ b/src/components/Activities/Regularization.jsx @@ -92,8 +92,8 @@ const Regularization = ({ handleRequest }) => { )} */} {!loading && - (regularizes?.length > 0 ? ( - regularizes?.map((att, index) => ( + (currentItems?.length > 0 ? ( + currentItems?.map((att, index) => (
From 2425387d8e97e160c923e3f3605b3c5dd0d4edaa Mon Sep 17 00:00:00 2001 From: pramod mahajan Date: Wed, 16 Jul 2025 18:30:06 +0530 Subject: [PATCH 02/12] display correctly activie,incative and all employee according user action --- src/hooks/useEmployees.js | 16 +++++++++------- src/pages/employee/EmployeeList.jsx | 12 +++--------- 2 files changed, 12 insertions(+), 16 deletions(-) diff --git a/src/hooks/useEmployees.js b/src/hooks/useEmployees.js index 3978ac81..2cbb9a75 100644 --- a/src/hooks/useEmployees.js +++ b/src/hooks/useEmployees.js @@ -112,18 +112,20 @@ export const useEmployeesByProject = (projectId) => { }; // EmployeeList.jsx -export const useEmployeesAllOrByProjectId = (projectId, showInactive) => { - const isAllEmployees = !projectId && projectId !== undefined; +export const useEmployeesAllOrByProjectId = (showAllEmployees ,projectId, + showInactive) => { - const queryKey = isAllEmployees - ? ['allEmployees', showInactive] - : ['projectEmployees', projectId]; + + const queryKey = showAllEmployees + ? ['allEmployees', showInactive] + : ['projectEmployees', projectId, showInactive]; const queryFn = async () => { - if (isAllEmployees) { + if (showAllEmployees) { const res = await EmployeeRepository.getAllEmployeeList(showInactive); return res.data; } else { + if (!projectId) return []; const res = await EmployeeRepository.getEmployeeListByproject(projectId); return res.data; } @@ -137,7 +139,7 @@ export const useEmployeesAllOrByProjectId = (projectId, showInactive) => { } = useQuery({ queryKey, queryFn, - enabled: isAllEmployees || !!projectId, + enabled:typeof showInactive === "boolean" && (showAllEmployees || !!projectId), }); return { diff --git a/src/pages/employee/EmployeeList.jsx b/src/pages/employee/EmployeeList.jsx index 09a820da..f04ec0cd 100644 --- a/src/pages/employee/EmployeeList.jsx +++ b/src/pages/employee/EmployeeList.jsx @@ -51,7 +51,7 @@ const EmployeeList = () => { const { employees, loading, setLoading, error, recallEmployeeData } = useEmployeesAllOrByProjectId( - showAllEmployees ? null : selectedProjectId, + showAllEmployees ,selectedProjectId, showInactive ); @@ -153,13 +153,7 @@ const EmployeeList = () => { } }; - const handleToggle = (e) => { - setShowInactive(e.target.checked); - recallEmployeeData( - e.target.checked, - showAllEmployees ? null : selectedProjectId - ); // Use selectedProjectId here - }; + const handleAllEmployeesToggle = (e) => { const isChecked = e.target.checked; @@ -340,7 +334,7 @@ const EmployeeList = () => { role="switch" id="inactiveEmployeesCheckbox" checked={showInactive} - onChange={handleToggle} + onChange={(e)=> setShowInactive(e.target.checked)} />