Skip to content

Drop docs redirect stubs from sidebar nav, redirect renamed doc paths - #5551

Merged
ZJvandeWeg merged 1 commit into
mainfrom
zj-docs-underscore-fixes
Aug 10, 2026
Merged

Drop docs redirect stubs from sidebar nav, redirect renamed doc paths#5551
ZJvandeWeg merged 1 commit into
mainfrom
zj-docs-underscore-fixes

Conversation

@ZJvandeWeg

Copy link
Copy Markdown
Member

Summary

  • Depends on docs: Remove underscore paths flowfuse#8154, which renames three underscore-named doc files (email_providers.md, user_management.md, aws_terraform.md) to fix nuxt-link-checker's no-underscores warning.
  • nuxt/lib/docs-nav.mjs now excludes any docs page with redirect frontmatter (e.g. admin/licensing.md, community-support.md) from the sidebar entirely — linking to them just bounces through a redirect, and since the sidebar renders on every docs page, that warning was repeating ~194 times in link-checker output.
  • nuxt/redirects.ts gets 301s for the three renamed doc paths so existing bookmarks/links don't 404 once docs: Remove underscore paths flowfuse#8154 ships.
  • .claude/CLAUDE.md documents that redirects belong in nuxt/redirects.ts (Nitro route rules), not netlify.toml (legacy, reserved for domain-alias redirects).

Test plan

  • npm run build:nuxt locally and confirm /docs/admin/licensing/ and /docs/community-support/ no longer appear in the docs sidebar
  • Confirm the sidebar still renders correctly for a normal section (e.g. Admin, Support)
  • Once docs: Remove underscore paths flowfuse#8154 merges and this branch is rebuilt, confirm /docs/install/email_providers/, /docs/admin/user_management/, /docs/install/kubernetes/aws_terraform/ 301 to their new paths

@ZJvandeWeg

Copy link
Copy Markdown
Member Author

Depends on FlowFuse/flowfuse#8154 — the redirect targets (/docs/install/email-providers/, /docs/admin/user-management/, /docs/install/kubernetes/aws-terraform/) only exist once that one merges and a docs rebuild picks it up. Merge order: flowfuse#8154 first, then this.

@netlify

netlify Bot commented Aug 9, 2026

Copy link
Copy Markdown

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit 5d7fcd5
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/6a7a1e6b7c294e0008e27509
😎 Deploy Preview https://deploy-preview-5551--flowforge-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 64 (🔴 down 3 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 85 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

FlowFuse/flowfuse#8154 renames three underscore-named doc files to fix
nuxt-link-checker's no-underscores warning, which was repeating on nearly
every docs page via the shared sidebar. Also excludes redirect-only doc
pages (Licensing, Community Support) from the sidebar entirely, since
linking to them just bounces through a redirect and triggered the same
warning on every page.

Adds 301s in nuxt/redirects.ts for the old underscore paths so existing
links/bookmarks don't 404, and documents in CLAUDE.md that redirects
belong in nuxt/redirects.ts, not netlify.toml.
@ZJvandeWeg
ZJvandeWeg force-pushed the zj-docs-underscore-fixes branch from a03a7a0 to 5d7fcd5 Compare August 10, 2026 18:54
@ZJvandeWeg
ZJvandeWeg merged commit 4892f23 into main Aug 10, 2026
5 checks passed
@ZJvandeWeg
ZJvandeWeg deleted the zj-docs-underscore-fixes branch August 10, 2026 18:55
@ZJvandeWeg

Copy link
Copy Markdown
Member Author

Merging prior to tests finishing as locally this is fine and fixes redirect issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant