fix: Fix PR comments

src/components/SidebarActions/SidebarActions.tsx

@@ -16,6 +16,7 @@ import DirectMessageModal from "./DirectMessageModal"
 import NotificationsModal from "./NotificationsModal"
 import Modal from "../Modal"
 import useNotifications from "./useNotifications"
+import {useNotificationsStateStore} from "./useCachedNotifications"
 
 const SidebarModalsHandler: FC = () => {
   const {sidebarModalActive} = useSidebarModalActiveStore()
@@ -58,8 +59,8 @@ export type SidebarActionsProps = {
 }
 
 const SidebarActions: FC<SidebarActionsProps> = ({className}) => {
-  const {onLogout, setActiveSidebarModal, containsUnreadNotifications} =
-    useSidebarActions()
+  const {containsUnreadNotifications} = useNotificationsStateStore()
+  const {onLogout, setActiveSidebarModal} = useSidebarActions()
 
   return (
     <>

src/components/SidebarActions/useCachedNotifications.ts

@@ -38,6 +38,7 @@ const useCachedNotifications = () => {
   })
 
   useEffect(() => {
+    // Check if you have unread notifications with the `some` method and it will refresh the global status of notifications.
     refreshContainsUnreadNotifications(
       cachedNotifications.some(notification => !notification.isRead)
     )

src/components/SidebarActions/useSidebarActions.ts

@@ -3,7 +3,6 @@ import {useNavigate} from "react-router-dom"
 import {ViewPath} from "@/utils/util"
 import {useCallback} from "react"
 import {create} from "zustand"
-import {useNotificationsStateStore} from "./useCachedNotifications"
 
 export enum SidebarModals {
   DirectMessages,
@@ -30,7 +29,6 @@ const useSidebarActions = () => {
   const {disconnect} = useConnection()
   const navigate = useNavigate()
   const {setActiveSidebarModal} = useSidebarModalActiveStore()
-  const {containsUnreadNotifications} = useNotificationsStateStore()
 
   const onLogout = useCallback(async () => {
     await disconnect()
@@ -40,7 +38,6 @@ const useSidebarActions = () => {
   return {
     onLogout,
     setActiveSidebarModal,
-    containsUnreadNotifications,
   }
 }
 

src/utils/util.ts

@@ -39,7 +39,7 @@ export function timeFormatter(timestamp: number): string {
 }
 
 export enum CommonAssertion {
-  EventIdNotFound = "eventId should not be undefined",
+  EventIdNotFound = "To confirm that an event happened, event id should not be undefined.",
 }
 
 export function assert(