Refactor NotificationContext to Streamline Notification Fetching Logic

- Removed the useEffectEvent for fetching notifications and integrated the fetching logic directly into the useEffect hooks for improved clarity and performance.
- Updated dependency arrays to ensure notifications are fetched correctly based on authentication and connection status, enhancing responsiveness to state changes.

src/components/Dashboard/context/NotificationContext.jsx

@@ -3,8 +3,7 @@ import {
   useState,
   useContext,
   useCallback,
-  useEffect,
+  useEffect
-  useEffectEvent
 } from 'react'
 import { useLocation } from 'react-router-dom'
 import { notification, Drawer } from 'antd'
@@ -60,11 +59,6 @@ const NotificationProvider = ({ children }) => {
     }
   }, [authenticated, fetchNotificationsApi, showError])
 
-  const onFetchNotifications = useEffectEvent(() => {
-    if (!authenticated || !connected) return
-    fetchNotifications()
-  })
-
   const markNotificationAsRead = useCallback(
     async (notificationId) => {
       try {
@@ -121,15 +115,15 @@ const NotificationProvider = ({ children }) => {
 
   // Initial load / when we become authenticated and connected
   useEffect(() => {
-    onFetchNotifications()
+    if (!authenticated || !connected) return
-  }, [authenticated, connected])
+    fetchNotifications()
+  }, [authenticated, connected, fetchNotifications])
 
   // Refresh when the notification center opens
   useEffect(() => {
-    if (notificationCenterVisible) {
+    if (!notificationCenterVisible || !authenticated || !connected) return
-      onFetchNotifications()
+    fetchNotifications()
-    }
+  }, [notificationCenterVisible, authenticated, connected, fetchNotifications])
-  }, [notificationCenterVisible])
 
   useEffect(() => {
     setNotificationCenterVisible(false)