From b2f1c3fbb045666a09716dbdb111445d21acf9a2 Mon Sep 17 00:00:00 2001
From: yadhap Dahal <yadhap.dahal@lexisnexisrisk.com>
Date: Tue, 7 Jan 2025 09:24:29 -0500
Subject: [PATCH 1/5] Changed key for the options in notification filter as it
 was throwing warning saying some of the keys are  not unique

---
 .../notifications/NotificationTableFilters.jsx       | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/NotificationTableFilters.jsx b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/NotificationTableFilters.jsx
index b287c40a..5df54cce 100644
--- a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/NotificationTableFilters.jsx
+++ b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/NotificationTableFilters.jsx
@@ -92,8 +92,8 @@ function NotificationTableFilters({ setFilters, sentNotifications, monitorings,
             <div className="notifications__filter-label">Status</div>
             <Form.Item name="status">
               <Select placeholder="Status" disabled={false} allowClear>
-                {statusOptions.map((s) => (
-                  <Option key={s} value={s}>
+                {statusOptions.map((s, i) => (
+                  <Option key={i} value={s}>
                     {s}
                   </Option>
                 ))}
@@ -106,8 +106,8 @@ function NotificationTableFilters({ setFilters, sentNotifications, monitorings,
                 <div className="notifications__filter-label">Domain</div>
                 <Form.Item name="domain">
                   <Select placeholder="Domain" disabled={false} allowClear>
-                    {domainOptions.map((d) => (
-                      <Option key={d} value={d}>
+                    {domainOptions.map((d, i) => (
+                      <Option key={i} value={d}>
                         {d}
                       </Option>
                     ))}
@@ -118,8 +118,8 @@ function NotificationTableFilters({ setFilters, sentNotifications, monitorings,
                 <div className="notifications__filter-label">Product</div>
                 <Form.Item name="product">
                   <Select placeholder="Product" disabled={false} allowClear>
-                    {productOptions.map((p) => (
-                      <Option key={p} value={p}>
+                    {productOptions.map((p, i) => (
+                      <Option key={i} value={p}>
                         {p}
                       </Option>
                     ))}

From 7c14f50ab4211ebb8addcfbcb63fc3fe712bb9e9 Mon Sep 17 00:00:00 2001
From: yadhap Dahal <yadhap.dahal@lexisnexisrisk.com>
Date: Tue, 7 Jan 2025 10:08:42 -0500
Subject: [PATCH 2/5] Added condition to hide or show Notifications by Product
 with Status Count based on if ASR is present or not

---
 .../notifications/NotificationDashboard.jsx   | 20 ++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/NotificationDashboard.jsx b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/NotificationDashboard.jsx
index 2684de74..076ebb78 100644
--- a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/NotificationDashboard.jsx
+++ b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/NotificationDashboard.jsx
@@ -1,5 +1,6 @@
 // Package imports
 import React, { useState, useEffect } from 'react';
+import { useSelector } from 'react-redux';
 import { Empty } from 'antd';
 
 // Local Imports
@@ -10,6 +11,9 @@ import NotificationCountByOriginDonut from './charts/NotificationCountByOriginDo
 
 function NotificationDashboard({ sentNotifications, dashBoardFilter, monitorings, productCategories }) {
   const [filteredNotifications, setFilteredNotifications] = useState([]);
+  const { applicationReducer } = useSelector((state) => state);
+  const { integrations: allIntegrations = [] } = applicationReducer;
+  const integrationCodes = allIntegrations.map((integration) => integration.name);
 
   useEffect(() => {
     console.log('Loading ...');
@@ -67,13 +71,15 @@ function NotificationDashboard({ sentNotifications, dashBoardFilter, monitorings
             <NotificationCountOnLineGraph sentNotifications={filteredNotifications} dashBoardFilter={dashBoardFilter} />
           </div>
 
-          <div className="notifications_chart_card">
-            <div className="notifications_chart_title">Notifications by Product with Status Count</div>
-            <NotificationCountByProductsInGraph
-              sentNotifications={filteredNotifications}
-              productCategories={productCategories}
-            />
-          </div>
+          {integrationCodes.includes('ASR') && (
+            <div className="notifications_chart_card">
+              <div className="notifications_chart_title">Notifications by Product with Status Count</div>
+              <NotificationCountByProductsInGraph
+                sentNotifications={filteredNotifications}
+                productCategories={productCategories}
+              />
+            </div>
+          )}
         </>
       )}
     </div>

From 79ff467db6d29659aab55e956e491040dfd48e95 Mon Sep 17 00:00:00 2001
From: yadhap Dahal <yadhap.dahal@lexisnexisrisk.com>
Date: Tue, 7 Jan 2025 10:13:17 -0500
Subject: [PATCH 3/5] Notification update was failing because the component was
 pulling user data from redux, user is in local storage now

---
 .../dashboard/notifications/UpdateNotification.jsx         | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/UpdateNotification.jsx b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/UpdateNotification.jsx
index b3585b69..65f7a544 100644
--- a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/UpdateNotification.jsx
+++ b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/UpdateNotification.jsx
@@ -1,6 +1,5 @@
 // Packages
 import React, { useState, useEffect } from 'react';
-import { useSelector } from 'react-redux';
 import { DownOutlined } from '@ant-design/icons';
 import {
   Modal,
@@ -47,7 +46,8 @@ const UpdateNotificationModal = ({
   const [warningMessages, setWarningMessages] = useState([]);
 
   //Redux
-  const { user } = useSelector((state) => state.authenticationReducer);
+  // Get user from local stroage
+  const user = JSON.parse(localStorage.getItem('user'));
 
   // Effects
   useEffect(() => {
@@ -185,8 +185,7 @@ const UpdateNotificationModal = ({
         setSelectedNotificationsIds([]);
       }
     } catch (err) {
-      console.error(err);
-      message.error('Enter updated information');
+      message.error('Failed to save updated notification(s)');
     } finally {
       form.resetFields();
     }

From 2054bd72fe3b2803c0c1c0c17b67e399b784331d Mon Sep 17 00:00:00 2001
From: yadhap Dahal <yadhap.dahal@lexisnexisrisk.com>
Date: Tue, 7 Jan 2025 10:17:32 -0500
Subject: [PATCH 4/5] Notifications nested differently now causing the
 Notifications by Product with Status Count graphnot rendering properly.fixed
 that issue

---
 .../charts/NotificationCountByProductsInGraph.jsx             | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/charts/NotificationCountByProductsInGraph.jsx b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/charts/NotificationCountByProductsInGraph.jsx
index 103b36d1..35b5814f 100644
--- a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/charts/NotificationCountByProductsInGraph.jsx
+++ b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/charts/NotificationCountByProductsInGraph.jsx
@@ -6,8 +6,8 @@ function NotificationCountByProductsInGraph({ sentNotifications, productCategori
   // Prepare the data
   let chartData = [];
   sentNotifications.forEach((notification) => {
-    if (notification.metaData && notification.metaData.asrSpecificMetaData) {
-      const category = notification.metaData.asrSpecificMetaData.productCode || 'ACU';
+    if (notification.metaData && notification.metaData.notificationDetails?.product) {
+      const category = notification.metaData.notificationDetails?.product;
       const status = notification.status;
       chartData.push({ category, status, count: 1 });
     }

From 8b126df1172ff473f2e152a2143b8e07a4bd2704 Mon Sep 17 00:00:00 2001
From: yadhap Dahal <yadhap.dahal@lexisnexisrisk.com>
Date: Wed, 8 Jan 2025 14:26:57 -0500
Subject: [PATCH 5/5] used centerlized get user function to retrive user
 information

---
 .../application/dashboard/notifications/UpdateNotification.jsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/UpdateNotification.jsx b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/UpdateNotification.jsx
index 65f7a544..02a9e563 100644
--- a/Tombolo/client-reactjs/src/components/application/dashboard/notifications/UpdateNotification.jsx
+++ b/Tombolo/client-reactjs/src/components/application/dashboard/notifications/UpdateNotification.jsx
@@ -21,6 +21,7 @@ import dayjs from 'dayjs';
 //Local Imports
 import { statuses } from './notificationUtil';
 import { updateMultipleNotifications } from './notificationUtil';
+import { getUser } from '../../../common/userStorage';
 
 const { Option } = Select;
 const { TextArea } = Input;
@@ -47,7 +48,7 @@ const UpdateNotificationModal = ({
 
   //Redux
   // Get user from local stroage
-  const user = JSON.parse(localStorage.getItem('user'));
+  const user = getUser();
 
   // Effects
   useEffect(() => {