chore: update sidebar logic

openedx · Nov 20, 2023 · f493a52 · f493a52
1 parent 748e73d
commit f493a52
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/courseware/course/sidebar/SidebarContextProvider.jsx b/src/courseware/course/sidebar/SidebarContextProvider.jsx
@@ -22,7 +22,13 @@ const SidebarProvider = ({
   const [upgradeNotificationCurrentState, setUpgradeNotificationCurrentState] = useState(getLocalStorage(`upgradeNotificationCurrentState.${courseId}`));
 
   useEffect(() => {
-    setCurrentSidebar(SIDEBARS.DISCUSSIONS.ID);
+    // if the user left the tray open in the previous session, open it again
+    // if the user is not verified, open the tray iff the previous session was open and the user on large screen
+    const openBySession = getSessionStorage(`notificationTrayStatus.${courseId}`) !== 'closed';
+    const showNotificationsOnLoad = openBySession || (!!verifiedMode && shouldDisplaySidebarOpen && openBySession );
+    if (showNotificationsOnLoad) {
+      setCurrentSidebar(SIDEBARS.NOTIFICATIONS.ID);
+    }
   // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [unitId]);
 

diff --git a/src/data/sessionStorage.js b/src/data/sessionStorage.js
@@ -7,10 +7,7 @@
 function getSessionStorage(key) {
   try {
     if (global.sessionStorage) {
-      const rawItem = global.sessionStorage.getItem(key);
-      if (rawItem) {
-        return JSON.parse(rawItem);
-      }
+      return global.sessionStorage.getItem(key);
     }
   } catch (e) {
     // If this fails for some reason, just return null.