fix: fixed empty section issue (openedx#1292)

* fix: fixed empty section issue * refactor: added checks to hide view
eduNEXT · Feb 15, 2024 · 9dd79ca · 9dd79ca
1 parent 9bfa0d0
commit 9dd79ca
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/courseware/course/new-sidebar/Sidebar.jsx b/src/courseware/course/new-sidebar/Sidebar.jsx
@@ -4,9 +4,9 @@ import SidebarContext from './SidebarContext';
 import { SIDEBARS } from './sidebars';
 
 const Sidebar = () => {
-  const { currentSidebar } = useContext(SidebarContext);
+  const { currentSidebar, isDiscussionbarAvailable, isNotificationbarAvailable } = useContext(SidebarContext);
 
-  if (currentSidebar === null) { return null; }
+  if (currentSidebar === null || (!isDiscussionbarAvailable && !isNotificationbarAvailable)) { return null; }
   const SidebarToRender = SIDEBARS[currentSidebar].Sidebar;
 
   return (

diff --git a/src/courseware/course/new-sidebar/SidebarContextProvider.jsx b/src/courseware/course/new-sidebar/SidebarContextProvider.jsx
@@ -32,7 +32,7 @@ const SidebarProvider = ({
   );
   const topic = useModel('discussionTopics', unitId);
   const { verifiedMode } = useModel('courseHomeMeta', courseId);
-  const isDiscussionbarAvailable = topic?.id && topic?.enabledInContext;
+  const isDiscussionbarAvailable = (topic?.id && topic?.enabledInContext) || false;
   const isNotificationbarAvailable = !isEmpty(verifiedMode);
 
   const onNotificationSeen = useCallback(() => {