Skip to content

πŸ“– docs: fix 404 on Reference section and audit every sidebar link - #225

Merged
clubanderson merged 1 commit into
mainfrom
docs-fix-sidebar-reference-404
Oct 5, 2026
Merged

clubanderson merged 1 commit into
mainfrom
docs-fix-sidebar-reference-404

Conversation

@clubanderson

Copy link
Copy Markdown
Member

Summary

  • Fix sidebar section labels so nested folders resolve to the first real leaf page instead of an intermediate folder route.
  • Add a regression test covering nested folders that carry their own route and children.
  • Audited the live docs sidebar on https://docs.hivecommons.dev/docs/hive/overview/introduction and verified the local fixed sidebar has zero broken links.

Live sidebar audit before fix

Link text URL Status Cause
Reference https://docs.hivecommons.dev/docs/hive/reference/architecture-decision-records 404 The rendered Reference folder label used the route of its first child folder (Architecture Decision Records) instead of recursing to that folder's first leaf page (ADR index). No page is generated for the intermediate folder route.

All other 72 rendered left-sidebar links returned 200, including section-header links for Community, Overview, Getting started, Operations, Integrating with Hive, Security, Architecture Decision Records, and sibling project entries.

After fix

Link text Fixed URL Local status
Reference http://127.0.0.1:3210/docs/hive/reference/architecture-decision-records/adr-index 200

Local rendered sidebar audit: 73 links checked, 0 broken.

Validation

  • npm run test -- DocsSidebar.behavior.test.tsx βœ…
  • npm run check-links βœ…
  • npm run lint βœ… (warnings only, pre-existing)
  • npm run build βœ…
  • Local production server audit of every rendered sidebar href: 73 checked, 0 broken βœ…

Signed-off-by: Andrew Anderson <andy@clubanderson.com>
@kubestellar-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign clubanderson for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@netlify

netlify Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

βœ… Deploy Preview for hivecommons-docs ready!

Name Link
πŸ”¨ Latest commit 4fed2f8
πŸ” Latest deploy log https://app.netlify.com/projects/hivecommons-docs/deploys/6ac408125b2fe8000892584f
😎 Deploy Preview https://deploy-preview-225--hivecommons-docs.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@clubanderson
clubanderson merged commit 00ff35a into main Oct 5, 2026
10 of 11 checks passed
@kubestellar-prow
kubestellar-prow Bot deleted the docs-fix-sidebar-reference-404 branch October 5, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant