fix (frontend): The notifications card doesn't need to be shown, when…

… there are no notifications.
UvA-FNWI · Oct 1, 2024 · 8118deb · 8118deb
1 parent 9ddbaee
commit 8118deb
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/IguideME.Web/Frontend/src/components/particles/notifications/notifications.tsx b/IguideME.Web/Frontend/src/components/particles/notifications/notifications.tsx
@@ -23,6 +23,16 @@ const Notifications: FC = (): ReactElement | null => {
     return null;
   }
 
+  // If there are no notifications, we don't want to render the card
+  if (
+    notifications.outperforming.length === 0 &&
+    notifications.closing.length === 0 &&
+    notifications.falling.length === 0 &&
+    notifications.effort.length === 0
+  ) {
+    return null;
+  }
+
   const styledNotifications = [
     notifications.outperforming.length > 0 && (
       <div key='outperforming'>

diff --git a/charts/iguideme/Chart.yaml b/charts/iguideme/Chart.yaml
@@ -2,5 +2,5 @@ apiVersion: v2
 name: iguideme
 description: IguideME
 type: application
-version: 0.3.222
-appVersion: "0.3.222"
+version: 0.3.223
+appVersion: "0.3.223"