Skip to content

fix: resolve 404 on API docs navigation link - #1008

Merged
IEvangelist merged 1 commit into
bradygaster:devfrom
ilyas-it83:squad/990-fix-api-docs-404
Apr 20, 2026
Merged

fix: resolve 404 on API docs navigation link#1008
IEvangelist merged 1 commit into
bradygaster:devfrom
ilyas-it83:squad/990-fix-api-docs-404

Conversation

@ilyas-it83

Copy link
Copy Markdown
Contributor

Summary

Fixes the 404 at https://bradygaster.github.io/squad/docs/reference/api/ reported in #990.

Root Cause

The navigation slug in docs/src/navigation.ts was reference/api, but the actual content file is api-reference.md which generates the slug reference/api-reference. Additionally, sdk.md had a broken absolute link to /reference/api/.

Changes

  • docs/src/navigation.ts — Fixed slug from reference/apireference/api-reference
  • docs/src/content/docs/reference/sdk.md — Changed broken absolute link to a relative markdown link (api-reference.md)

Closes #990

Copilot AI review requested due to automatic review settings April 19, 2026 15:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes broken API documentation navigation/linking in the Astro docs site so the “SDK API Reference” entry no longer routes to a non-existent slug on GitHub Pages.

Changes:

  • Updated the sidebar navigation slug for “SDK API Reference” to reference/api-reference to match the generated route for api-reference.md.
  • Replaced a broken absolute link in sdk.md with a relative markdown link to api-reference.md (compatible with the repo’s link-rewrite plugin).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/src/navigation.ts Aligns the “SDK API Reference” nav slug with the actual generated page route.
docs/src/content/docs/reference/sdk.md Fixes a broken API reference link by switching to a relative .md link.

The navigation slug 'reference/api' did not match the actual content file
'api-reference.md' (which generates slug 'reference/api-reference'). Fixed
the slug in navigation.ts and updated the broken absolute link in sdk.md
to use a relative markdown link instead.

Closes bradygaster#990

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ilyas-it83
ilyas-it83 force-pushed the squad/990-fix-api-docs-404 branch from 4f12feb to f7f7037 Compare April 19, 2026 15:45

@IEvangelist IEvangelist left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@IEvangelist
IEvangelist merged commit a3ba665 into bradygaster:dev Apr 20, 2026
tamirdresher pushed a commit that referenced this pull request Apr 21, 2026
fix: resolve 404 on API docs navigation link
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.

404 on API docs

3 participants