Skip to content

fix: SolidStart v2 routing: sidebar and inline links#1570

Merged
LadyBluenotes merged 2 commits into
solidjs:mainfrom
tknkaa:fix/solid-start-v2-routing
Jul 6, 2026
Merged

fix: SolidStart v2 routing: sidebar and inline links#1570
LadyBluenotes merged 2 commits into
solidjs:mainfrom
tknkaa:fix/solid-start-v2-routing

Conversation

@tknkaa

@tknkaa tknkaa commented Jul 5, 2026

Copy link
Copy Markdown
Contributor
  • I have read the Contribution guide
  • This PR references an issue (except for typos, broken links, or other minor problems)

Description(required)

Three routing bugs affecting the SolidStart v2 documentation:

  1. The start, latest sidebar was built without a filter, causing the v2/ subdirectory to appear as a collapsible "V2" section in the Latest SolidStart navigation.
  2. The start, v2 sidebar was registered under the wrong key (/v2/solid-start instead of /solid-start/v2), producing broken prev/next navigation links on v2 pages.
  3. All inline cross-links in 12 v2 MDX files used the wrong URL pattern (/v2/solid-start/... instead of /solid-start/v2/...), resulting in 404s.

Related issues & labels

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify

netlify Bot commented Jul 5, 2026

Copy link
Copy Markdown

Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit 8c0a832
🔍 Latest deploy log https://app.netlify.com/projects/solid-docs/deploys/6a4a558ae8296c0008f3b1bd
😎 Deploy Preview https://deploy-preview-1570--solid-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@LadyBluenotes

Copy link
Copy Markdown
Member

@tknkaa Thank you for your PR working on this. Similar to the previous PR, can you please add back in the PR template and indicate whether you've read the contributing guidelines?

@tknkaa

tknkaa commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

@LadyBluenotes
Fixed. Can you review these PRs?

@LadyBluenotes LadyBluenotes merged commit fec1d6b into solidjs:main Jul 6, 2026
8 checks passed
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.

[Bug]: SolidStart v2 section leaks into Latest sidebar and inline links 404

2 participants