fix: remove redundant fetchNotifications() from onMount in NotificationBell
Some checks failed
CI / Unit & Component Tests (pull_request) Has been cancelled
CI / Backend Unit Tests (pull_request) Has been cancelled
CI / E2E Tests (pull_request) Has been cancelled
CI / Unit & Component Tests (push) Successful in 2m39s
CI / Backend Unit Tests (push) Successful in 2m21s
CI / E2E Tests (push) Has started running
Some checks failed
CI / Unit & Component Tests (pull_request) Has been cancelled
CI / Backend Unit Tests (pull_request) Has been cancelled
CI / E2E Tests (pull_request) Has been cancelled
CI / Unit & Component Tests (push) Successful in 2m39s
CI / Backend Unit Tests (push) Successful in 2m21s
CI / E2E Tests (push) Has started running
Notifications are already fetched lazily inside toggleDropdown() when the user opens the dropdown. Only fetchUnreadCount() is needed on mount to show the badge. Closes #725 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit was merged in pull request #127.
This commit is contained in:
@@ -144,7 +144,6 @@ function relativeTime(isoString: string): string {
|
|||||||
}
|
}
|
||||||
|
|
||||||
onMount(() => {
|
onMount(() => {
|
||||||
fetchNotifications();
|
|
||||||
fetchUnreadCount();
|
fetchUnreadCount();
|
||||||
eventSource = new EventSource('/api/notifications/stream');
|
eventSource = new EventSource('/api/notifications/stream');
|
||||||
eventSource.addEventListener('notification', (e) => {
|
eventSource.addEventListener('notification', (e) => {
|
||||||
|
|||||||
Reference in New Issue
Block a user