Skip to content

Commit 78e4910

Browse files
committed
Address CR
1 parent a6c195c commit 78e4910

File tree

1 file changed

+14
-12
lines changed

1 file changed

+14
-12
lines changed

apps/web/components/roadmap/hooks/useRoadmapItems.ts

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -67,25 +67,27 @@ export function useRoadmapItems({
6767
if (!confirm("Are you sure you want to delete this item?")) return;
6868

6969
try {
70+
const itemToDelete = Object.values(itemsByColumn)
71+
.flat()
72+
.find((it) => it.id === itemId);
73+
7074
const { error } = await supabase
7175
.from("roadmap_items")
7276
.delete()
7377
.eq("id", itemId);
7478

7579
if (error) throw error;
7680

77-
setBoardItems((prev) => {
78-
const deletedItem = prev.find((item) => item.id === itemId);
79-
if (deletedItem && user) {
80-
createAuditLog(supabase, {
81-
page_id: board.page_id,
82-
actor_id: user.id,
83-
action: `Deleted Roadmap Item: ${deletedItem.title}`,
84-
changes: { item: deletedItem },
85-
});
86-
}
87-
return prev.filter((item) => item.id !== itemId);
88-
});
81+
setBoardItems((prev) => prev.filter((item) => item.id !== itemId));
82+
83+
if (itemToDelete && user) {
84+
await createAuditLog(supabase, {
85+
page_id: board.page_id,
86+
actor_id: user.id,
87+
action: `Deleted Roadmap Item: ${itemToDelete.title}`,
88+
changes: { item_id: itemToDelete.id, item_title: itemToDelete.title },
89+
});
90+
}
8991
} catch (error) {
9092
console.error("Error deleting item:", error);
9193
alert("Failed to delete item");

0 commit comments

Comments
 (0)