fix: remove unnecessary cleanup in useEffect hooks

anuj452005 · anuj452005 · commit f1319f3d48c5 · 2025-08-21T03:21:47.000+05:30
diff --git a/src/components/TaskApprovementPanel.jsx b/src/components/TaskApprovementPanel.jsx
@@ -92,7 +92,6 @@ function TaskApprovementPanel({
         );
       }
     }
-    return () => confirmed && clearConfirm(false);
   }, [confirmed]);
 
   const openResolveTaskConfirmDialog = (choiceString) => coreConfirm(
diff --git a/src/components/groups-management/TaskGroupsSearcher.jsx b/src/components/groups-management/TaskGroupsSearcher.jsx
@@ -70,7 +70,6 @@ function TaskGroupsSearcher({
     if (taskGroupToDelete && confirmed !== null) {
       setTaskGroupToDelete(null);
     }
-    return () => confirmed && clearConfirm(false);
   }, [confirmed]);
 
   useEffect(() => {

Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,6 @@ function TaskApprovementPanel({`
`92`	`92`	`);`
`93`	`93`	`}`
`94`	`94`	`}`
`95`		`- return () => confirmed && clearConfirm(false);`
`96`	`95`	`}, [confirmed]);`
`97`	`96`
`98`	`97`	`const openResolveTaskConfirmDialog = (choiceString) => coreConfirm(`
Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,6 @@ function TaskGroupsSearcher({`
`70`	`70`	`if (taskGroupToDelete && confirmed !== null) {`
`71`	`71`	`setTaskGroupToDelete(null);`
`72`	`72`	`}`
`73`		`- return () => confirmed && clearConfirm(false);`
`74`	`73`	`}, [confirmed]);`
`75`	`74`
`76`	`75`	`useEffect(() => {`