update for of loop with for each loop

Author: PraveenKum11

frontend/packages/app/src/pages/project-details/tabs/risks/kanban/kanbanView.tsx

@@ -34,27 +34,21 @@ export function RisksKanbanView() {
     (c) => c.actions.openCreateRiskWithStatus,
   );
   const toast = useToasts();
-  const [localData, setLocalData] = useState<RiskItem[]>(data);
-
-  useEffect(() => {
-    if (!data || data.length === 0) return;
-    setLocalData(data);
-  }, [data]);
 
   const [items, setItems] = useState<RiskIdsByStatus>(emptyGroups);
 
   const byId = useMemo(() => {
     const map = new Map<string, RiskItem>();
-    for (const risk of localData) {
+    for (const risk of data) {
       map.set(risk.name, risk);
     }
     return map;
-  }, [localData]);
+  }, [data]);
 
   useEffect(() => {
-    if (!localData) return;
-    setItems(groupIdsByStatus(localData));
-  }, [localData]);
+    if (!data) return;
+    setItems(groupIdsByStatus(data));
+  }, [data]);
 
   return (
     <DragDropProvider<RiskDragData, RiskDraggable, RiskDroppable>
@@ -78,7 +72,7 @@ export function RisksKanbanView() {
         try {
           await updateRiskStatus(riskId, newStatus);
         } catch {
-          setItems(groupIdsByStatus(localData));
+          setItems(groupIdsByStatus(data));
           const risk = byId.get(riskId);
           toast.error(
             `Error updating status for ${risk?.risk_category ?? riskId}`,

frontend/packages/app/src/pages/project-details/tabs/risks/list/listView.tsx

@@ -1,29 +1,19 @@
 /**
  * External dependencies.
  */
-import { useEffect, useState } from "react";
 import { Accordion } from "@base-ui/react/accordion";
 
 /**
  * Internal dependencies.
  */
 import { useRisks } from "../context";
 import { RiskGroup } from "./listViewGroup";
-import { RiskItem } from "../types";
 
 export function RisksListView() {
   const data = useRisks((c) => c.state.data);
-  const [localData, setLocalData] = useState<RiskItem[]>(data);
 
-  useEffect(() => {
-    if (!data || data.length === 0) return;
-    setLocalData(data);
-  }, [data]);
-
-  const openRisks = localData.filter(
-    (r) => !r.status || r.status !== "Mitigated",
-  );
-  const mitigatedRisks = localData.filter((r) => r.status === "Mitigated");
+  const openRisks = data.filter((r) => !r.status || r.status !== "Mitigated");
+  const mitigatedRisks = data.filter((r) => r.status === "Mitigated");
 
   return (
     <Accordion.Root multiple defaultValue={["open", "mitigated"]}>