From 05e299d9760c2e0a2d23841a0e970962bb8230c0 Mon Sep 17 00:00:00 2001
From: julianajlk <julianajlk@gmail.com>
Date: Tue, 29 Jun 2021 14:26:15 -0400
Subject: [PATCH] Deactivate notification red dot

---
 src/courseware/course/Course.jsx                   | 2 +-
 src/courseware/course/Course.test.jsx              | 2 +-
 src/courseware/course/NotificationTrigger.jsx      | 4 ++--
 src/courseware/course/NotificationTrigger.test.jsx | 4 ++--
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/courseware/course/Course.jsx b/src/courseware/course/Course.jsx
index 0c28937a..12db2b60 100644
--- a/src/courseware/course/Course.jsx
+++ b/src/courseware/course/Course.jsx
@@ -61,7 +61,7 @@ function Course({
     courseId, sequenceId, unitId, celebrateFirstSection, dispatch, celebrations,
   );
 
-  // REV-2130 TODO: temporary cookie code that should be removed.
+  // REV-2297 TODO: temporary cookie code that should be removed.
   // In order to see the Value Prop Notification Tray in prod, a cookie should be set in
   // the browser console and refresh: document.cookie = 'value_prop_cookie=true';
   const isValuePropCookieSet = Cookies.get('value_prop_cookie') === 'true';
diff --git a/src/courseware/course/Course.test.jsx b/src/courseware/course/Course.test.jsx
index d9d202fa..5b795e5f 100644
--- a/src/courseware/course/Course.test.jsx
+++ b/src/courseware/course/Course.test.jsx
@@ -91,7 +91,7 @@ describe('Course', () => {
     const toggleNotificationTray = jest.fn();
     const isNotificationTrayVisible = jest.fn();
 
-    // REV-2130 TODO: remove cookie related code once temporary value prop cookie code is removed.
+    // REV-2297 TODO: remove cookie related code once temporary value prop cookie code is removed.
     const cookieName = 'value_prop_cookie';
     Cookies.set = jest.fn();
     Cookies.get = jest.fn().mockImplementation(() => cookieName);
diff --git a/src/courseware/course/NotificationTrigger.jsx b/src/courseware/course/NotificationTrigger.jsx
index db47a056..183a35a1 100644
--- a/src/courseware/course/NotificationTrigger.jsx
+++ b/src/courseware/course/NotificationTrigger.jsx
@@ -14,8 +14,8 @@ function NotificationTrigger({ intl, toggleNotificationTray, isNotificationTrayV
       onClick={() => { toggleNotificationTray(); }}
       aria-label={intl.formatMessage(messages.openNotificationTrigger)}
     >
-      {/* REV-2130 TODO: add logic for status "active" if red dot should display */}
-      <NotificationIcon status="active" notificationColor="bg-danger-500" />
+      {/* REV-2297 TODO: add logic for status "active" if red dot should display */}
+      <NotificationIcon status="inactive" notificationColor="bg-danger-500" />
     </button>
   );
 }
diff --git a/src/courseware/course/NotificationTrigger.test.jsx b/src/courseware/course/NotificationTrigger.test.jsx
index 1ebf7439..918bb737 100644
--- a/src/courseware/course/NotificationTrigger.test.jsx
+++ b/src/courseware/course/NotificationTrigger.test.jsx
@@ -23,8 +23,8 @@ describe('Notification Trigger', () => {
     const buttonIcon = container.querySelectorAll('svg');
     expect(buttonIcon).toHaveLength(1);
 
-    // REV-2130 TODO: update below test once the status=active or inactive is implemented
-    expect(screen.getByTestId('notification-dot')).toBeInTheDocument();
+    // REV-2297 TODO: update below test once the status=active or inactive is implemented
+    // expect(screen.getByTestId('notification-dot')).toBeInTheDocument();
   });
 
   it('handles onClick event toggling the notification tray', async () => {