From 575e62f6dab44b0c9836623fe8b7d17a219f8e0f Mon Sep 17 00:00:00 2001 From: Christian Cleberg Date: Wed, 15 Jul 2026 21:56:56 -0500 Subject: fix: push patchset views directly instead of by route Tapping a patch failed with "no matching navigationDestination declaration visible from the location of the link". MailingListDetailView is presented from four places, but only the More tab and Lookup declare a MoreRoute destination. Reached from a project, via ProjectMailingListView, there is no such destination in the surrounding stack, so a NavigationLink carrying MoreRoute.patchset had nowhere to resolve. The thread rows beside it already use the closure form for exactly this reason. Push PatchsetDetailView directly, from the rows and from the version-chain links inside the detail view, which inherits whatever stack presented it. That leaves MoreRoute.patchset with no users, so it and its two destinations are removed rather than left as a route nothing links to. Neither the compiler nor the tests catch this: it is a runtime SwiftUI resolution failure. --- Hutch/Views/Projects/ProjectMailingListView.swift | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'Hutch/Views/Projects/ProjectMailingListView.swift') diff --git a/Hutch/Views/Projects/ProjectMailingListView.swift b/Hutch/Views/Projects/ProjectMailingListView.swift index 3933bf3..696cf6b 100644 --- a/Hutch/Views/Projects/ProjectMailingListView.swift +++ b/Hutch/Views/Projects/ProjectMailingListView.swift @@ -420,7 +420,11 @@ struct MailingListDetailView: View { if showingPatches(viewModel) { ForEach(viewModel.filteredPatchsets) { patchset in - NavigationLink(value: MoreRoute.patchset(id: patchset.id, listName: mailingList.name)) { + // Pushed directly rather than by value: this view is also shown + // from a project, whose stack declares no MoreRoute destination. + NavigationLink { + PatchsetDetailView(patchsetID: patchset.id, listName: mailingList.name) + } label: { PatchsetRow(patchset: patchset) } .themedRow() -- cgit v1.2.3