Deactivate notification red dot

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';

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);

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>
   );
 }

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 () => {