From 7ce43b09be3f7c611518f599b0acab844f4c0113 Mon Sep 17 00:00:00 2001
From: systemblue <systemblue@noreply.codeberg.org>
Date: Fri, 5 Jun 2026 22:01:31 -0400
Subject: [PATCH] fix: refresh the merged notifications list when a push
 notification is opened

Tapping a push notification bumps NavigationState's refresh trigger so
the notifications tab reloads on arrival. The single-instance overview
observes the trigger, but the merged multi-instance overview never did,
so multi-instance users landed on a stale list that still showed the
tapped thread as unread until a manual pull-to-refresh.

Observe the same trigger in the merged overview, mirroring
NotificationsOverviewView.
---
 Forji/Forji/Views/MergedNotificationsOverviewView.swift | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Forji/Forji/Views/MergedNotificationsOverviewView.swift b/Forji/Forji/Views/MergedNotificationsOverviewView.swift
index 3e4fdcd..807e3af 100644
--- a/Forji/Forji/Views/MergedNotificationsOverviewView.swift
+++ b/Forji/Forji/Views/MergedNotificationsOverviewView.swift
@@ -6,6 +6,7 @@ struct MergedNotificationsOverviewView: View {
 
     @State private var pagination: PaginationState<TaggedItem<NotificationThread>>
     @State private var statusFilter: String = "unread"
+    @Environment(NavigationState.self) private var navigationState
 
     init(manager: MultiInstanceManager) {
         self.manager = manager
@@ -95,6 +96,9 @@ struct MergedNotificationsOverviewView: View {
         .onChange(of: statusFilter) {
             reloadNotifications(clearItems: true)
         }
+        .onChange(of: navigationState.notificationsRefreshTrigger) {
+            reloadNotifications()
+        }
         .errorAlert(message: $pagination.errorMessage, isPresented: $pagination.showError)
     }