Skip to content

fix(NavigationManager): only resolve navigations of booted apps#61120

Draft
susnux wants to merge 2 commits into
masterfrom
fix/navigation-manager
Draft

fix(NavigationManager): only resolve navigations of booted apps#61120
susnux wants to merge 2 commits into
masterfrom
fix/navigation-manager

Conversation

@susnux

@susnux susnux commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Prevent log flooded with "route not found"

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@susnux susnux requested a review from come-nc June 9, 2026 16:07
@susnux susnux requested a review from a team as a code owner June 9, 2026 16:07
@susnux susnux requested review from leftybournes, provokateurin and salmart-dev and removed request for a team June 9, 2026 16:07
@susnux susnux added bug 3. to review Waiting for reviews labels Jun 9, 2026
@susnux susnux added this to the Nextcloud 35 milestone Jun 9, 2026
@come-nc

This comment was marked as resolved.

Comment thread lib/private/NavigationManager.php Outdated
@susnux susnux marked this pull request as draft June 10, 2026 15:42
@susnux

susnux commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

We discovered more issues, will work on it.

@susnux susnux force-pushed the fix/navigation-manager branch 3 times, most recently from 0ccc152 to b838ee0 Compare June 22, 2026 10:46
susnux added 2 commits June 22, 2026 20:37
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
…tion

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the fix/navigation-manager branch from 281ccd1 to e870c6f Compare June 22, 2026 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants