farmcontrol/farmcontrol-ui
farmcontrol/farmcontrol-ui
2
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Enhance NotificationProvider to fetch notifications based on authentication and token presence, improving notification handling logic.

Browse Source
This commit is contained in:
Tom Butcher 2026-06-21 19:42:01 +01:00
parent 128cf603e7
commit 6d839ebe9c
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/components/Dashboard/context/NotificationContext.jsx
View File

@ -19,7 +19,7 @@ const NotificationContext = createContext()
const NotificationProvider = ({ children }) => { const NotificationProvider = ({ children }) => {
const [api, contextHolder] = notification.useNotification() const [api, contextHolder] = notification.useNotification()
const location = useLocation() const location = useLocation()
const { authenticated } = useContext(AuthContext) const { authenticated, token } = useContext(AuthContext)
const { const {
showError, showError,
fetchNotificationsApi, fetchNotificationsApi,
@ -118,6 +118,12 @@ const NotificationProvider = ({ children }) => {
} }
}, [authenticated, notificationCenterVisible, fetchNotifications]) }, [authenticated, notificationCenterVisible, fetchNotifications])
useEffect(() => {
if (authenticated && token) {
fetchNotifications()
}
}, [authenticated, token, fetchNotifications])
useEffect(() => { useEffect(() => {
setNotificationCenterVisible(false) setNotificationCenterVisible(false)
}, [location.pathname]) }, [location.pathname])