Fix removeToastNotification to use previousState

cultureamp · Sep 20, 2024 · 4422137 · 4422137
1 parent c29094e
commit 4422137
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/.changeset/grumpy-wolves-promise.md b/.changeset/grumpy-wolves-promise.md
@@ -0,0 +1,5 @@
+---
+"@kaizen/components": patch
+---
+
+Fix removeToastNotification
diff --git a/packages/components/src/Notification/ToastNotification/context/ToastNotificationContext.tsx b/packages/components/src/Notification/ToastNotification/context/ToastNotificationContext.tsx
@@ -70,12 +70,7 @@ export const ToastNotificationProvider = ({
   }
 
   const removeToastNotification = (notificationID: string): void => {
-    const notificationIndex = notifications.findIndex(
-      ({ id }) => id === notificationID
-    )
-    const copy = notifications.slice()
-    copy.splice(notificationIndex, 1) // Mutation
-    setNotifications(copy)
+    setNotifications(prev => prev.filter(({ id }) => id !== notificationID))
   }
 
   const clearToastNotifications = (): void => {