Fix duplicate notifications when switching chains (#146)

ScopeLift · Dec 4, 2023 · 9cff14d · 9cff14d
1 parent 5d33d15
commit 9cff14d
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/contexts/NotificationsContext.tsx b/contexts/NotificationsContext.tsx
@@ -27,10 +27,14 @@ const NotificationsContext = createContext<NotificationsContextType>({
 export const NotificationsProvider = ({ children }: { children: ReactNode }) => {
   const [notifications, setNotifications] = useState<TxNotificationWithId[]>([]);
   const notify = (notification: TxNotification) => {
+    //if notification id already exists, don't add it again
+    const txId = `${notification.hash}-${notification.txStatus}-${notification.chainId}`;
+    if (notifications.some((n) => n.id === txId)) return;
+
     setNotifications([
       {
         ...notification,
-        id: `${notification.hash}-${notification.txStatus}-${notification.chainId}`,
+        id: txId,
       },
       ...notifications,
     ]);