feat(toast): use id for open status toggling (#178)

QuivrHQ · May 27, 2023 · 655f07d · 655f07d · vercel · May 27, 2023
1 parent e388990
commit 655f07d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/frontend/app/components/ui/Toast/components/Toast.tsx b/frontend/app/components/ui/Toast/components/Toast.tsx
@@ -18,12 +18,12 @@ export const Toast = ({
       <ToastContext.Provider value={{ publish }}>
         {children}
         <AnimatePresence mode="popLayout">
-          {toasts.map((toast, index) => {
+          {toasts.map((toast) => {
             if (!toast.open) return;
             return (
               <ToastPrimitive.Root
                 open={toast.open}
-                onOpenChange={(value) => toggleToast(value, index)}
+                onOpenChange={(value) => toggleToast(value, toast.id)}
                 asChild
                 forceMount
                 key={toast.id}

diff --git a/frontend/app/components/ui/Toast/hooks/useToastBuilder.ts b/frontend/app/components/ui/Toast/hooks/useToastBuilder.ts
@@ -6,10 +6,10 @@ import { generateToastUniqueId } from "../helpers/generateToastUniqueId";
 export const useToastBuilder = () => {
   const [toasts, setToasts] = useState<ToastContent[]>([]);
 
-  const toggleToast = (value: boolean, index: number) => {
+  const toggleToast = (value: boolean, toastId: string) => {
     setToasts((toasts) =>
-      toasts.map((toast, i) => {
-        if (i === index) {
+      toasts.map((toast) => {
+        if (toast.id === toastId) {
           toast.open = value;
         }
         return toast;