From 9df18da232c50ebeba5a85c025a3a53ad57b0ad7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=81d=C3=A1m=20Sis=C3=A1k?=
 <86593861+dzsak@users.noreply.github.com>
Date: Mon, 8 Apr 2024 16:17:52 +0200
Subject: [PATCH] Store dismissed events in local storage (#75)

---
 web/src/ToastNotifications.jsx | 33 +++++++++++++++++++++------------
 web/src/redux.js               |  9 ---------
 2 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/web/src/ToastNotifications.jsx b/web/src/ToastNotifications.jsx
index cdddacd..93eb0d1 100644
--- a/web/src/ToastNotifications.jsx
+++ b/web/src/ToastNotifications.jsx
@@ -1,5 +1,4 @@
-import { useState } from 'react';
-import { ACTION_DISMISS_FLUX_EVENT } from './redux';
+import { useState, useEffect } from 'react';
 import { XMarkIcon } from '@heroicons/react/24/outline';
 import { Transition } from '@headlessui/react'
 import { NavigationButton } from './NavigationButton';
@@ -9,23 +8,33 @@ function ToastNotifications(props) {
 
   let reduxState = store.getState();
   const [fluxEvents, setFluxEvents] = useState(reduxState.fluxEvents);
-  const [dismissedFluxEvents, setDismissedFluxEvents] = useState(reduxState.dismissedFluxEvents);
+  const [dismissed, setDismissed] = useState(JSON.parse(localStorage.getItem("dismissed")) ?? [])
   store.subscribe(() => setFluxEvents(reduxState.fluxEvents));
-  store.subscribe(() => setDismissedFluxEvents(reduxState.dismissedFluxEvents));
 
-  const warningEvents = fluxEvents.filter(e => e.type === "Warning" && !dismissedFluxEvents.some(de => isSameEvent(e, de))).slice(0, 3);
+  useEffect(() => {
+    localStorage.setItem("dismissed", JSON.stringify(dismissed));
+  }, [dismissed]);
+
+  useEffect(() => {
+    setDismissed(dismissed.filter(d => {
+      const parsed = Date.parse(d.firstTimestamp, "yyyy-MM-dd'T'HH:mm:ss");
+      const day = 24 * 60 * 60 * 1000;
+      return (Date.now() - parsed) < day
+    }))
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, []);
 
   const dismiss = (event) => {
-    store.dispatch({
-      type: ACTION_DISMISS_FLUX_EVENT, payload: event
-    });
+    setDismissed([...dismissed, event]);
   }
 
+  const warningEvents = fluxEvents.filter(e => e.type === "Warning" && !dismissed.some(d => isSameEvent(e, d))).slice(0, 3);
+
   return (
     <div className="fixed top-0 inset-x-0 z-50 text-center text-gray-700 text-sm space-y-2 max-w-7xl mx-auto py-2">
       {warningEvents.map((e, index) => {
         return (
-          <ToastElement key={index} event={e} index={index} dismiss={dismiss} handleNavigationSelect={handleNavigationSelect} />
+          <ToastElement key={index} event={e} dismiss={dismiss} handleNavigationSelect={handleNavigationSelect} />
         )
       })}
     </div>
@@ -33,7 +42,7 @@ function ToastNotifications(props) {
 }
 
 function ToastElement(props) {
-  const { event, index, dismiss, handleNavigationSelect } = props;
+  const { event, dismiss, handleNavigationSelect } = props;
   return (
     <Transition
       as="div"
@@ -48,10 +57,10 @@ function ToastElement(props) {
         leaveFrom="opacity-100 translate-y-0"
         leaveTo="opacity-0 -translate-y-12"
       >
-        <div key={index} className="rounded-md shadow-lg bg-orange-400" role="alert">
+        <div className="rounded-md shadow-lg bg-orange-400" role="alert">
           <div className="flex p-4">
             <NavigationButton handleNavigation={() => handleNavigationSelect(event.involvedObjectKind === "Kustomization" ? "Kustomizations" : "Sources", event.involvedObjectNamespace, event.involvedObject, event.involvedObjectKind)}>
-              <span className='font-bold'>{event.involvedObjectKind} {event.involvedObjectNamespace}/{event.involvedObject}</span>: {event.message}
+              <p className="break-all line-clamp-3"><span className='font-bold'>{event.involvedObjectKind} {event.involvedObjectNamespace}/{event.involvedObject}</span>: {event.message}</p>
             </NavigationButton>
             <div className="ml-auto">
               <button
diff --git a/web/src/redux.js b/web/src/redux.js
index c5c7891..40ace2a 100644
--- a/web/src/redux.js
+++ b/web/src/redux.js
@@ -6,7 +6,6 @@ import * as serviceEventHandlers from './serviceEventHandlers';
 export const initialState = {
   fluxState: {},
   fluxEvents: [],
-  dismissedFluxEvents: [],
   services: [],
   podLogs: {},
   textColors: {},
@@ -14,7 +13,6 @@ export const initialState = {
 
 export const ACTION_FLUX_STATE_RECEIVED = 'FLUX_STATE_RECEIVED';
 export const ACTION_FLUX_EVENTS_RECEIVED = 'FLUX_EVENTS_RECEIVED';
-export const ACTION_DISMISS_FLUX_EVENT = 'FLUX_EVENTS_DISMISSED';
 export const ACTION_SERVICES_RECEIVED = 'SERVICES_RECEIVED';
 export const ACTION_POD_LOGS_RECEIVED = 'POD_LOGS_RECEIVED';
 export const ACTION_CLEAR_PODLOGS = 'CLEAR_POD_LOGS';
@@ -41,8 +39,6 @@ export function rootReducer(state = initialState, action) {
       return fluxStateReceived(state, action.payload)
     case ACTION_FLUX_EVENTS_RECEIVED:
       return fluxEventsReceived(state, action.payload)
-    case ACTION_DISMISS_FLUX_EVENT:
-      return dismissFluxEvent(state, action.payload)
     case ACTION_SERVICES_RECEIVED:
       return servicesReceived(state, action.payload)
     case ACTION_POD_LOGS_RECEIVED:
@@ -92,11 +88,6 @@ function fluxEventsReceived(state, payload) {
   return state
 }
 
-function dismissFluxEvent(state, payload) {
-  state.dismissedFluxEvents = [...state.dismissedFluxEvents, payload]
-  return state
-}
-
 function servicesReceived(state, payload) {
   state.services = payload
   return state