frontend: ActionsNotifier: Use unique refKey for errors

This change adds a unique refKey for error states, separate from success states, addressing a regression where failure notifications would not appear in the UI. Fixes: #2655 Signed-off-by: Evangelos Skopelitis <[email protected]>
headlamp-k8s · Dec 12, 2024 · 913c272 · 913c272
1 parent b1e62cf
commit 913c272
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/frontend/src/components/common/ActionsNotifier.tsx b/frontend/src/components/common/ActionsNotifier.tsx
@@ -58,9 +58,13 @@ function PureActionsNotifier({ dispatch, clusterActions }: PureActionsNotifierPr
     }
 
     if (clusterAction.message) {
-      // Check for completed actions
+      // Check for success or error states
       const refKey =
-        clusterAction.state === 'complete' ? `${clusterAction.id}-complete` : clusterAction.id;
+        clusterAction.state === 'complete'
+          ? `${clusterAction.id}-complete`
+          : clusterAction.state === 'error'
+          ? `${clusterAction.id}-error`
+          : clusterAction.id;
 
       if (!snackbarRefs.current[refKey]) {
         snackbarRefs.current[refKey] = uniqueKey;