Skip to content

fix: bypass SPA routing for /llms.txt link#415

Merged
DovOps merged 2 commits into
mainfrom
copilot/fix-traderx-portall-llm-navigation
Jun 15, 2026
Merged

fix: bypass SPA routing for /llms.txt link#415
DovOps merged 2 commits into
mainfrom
copilot/fix-traderx-portall-llm-navigation

Conversation

Copilot AI commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Docusaurus's client-side Link (to prop) intercepts navigation to /llms.txt and attempts SPA routing — failing because it's a static file, not a React page. A hard browser refresh works because it bypasses the router entirely.

Changes

  • homepageData.js: Replace to: '/llms.txt' with href: '/llms.txt' on the LLM-Native Docs card so the SmartLink component falls through to a plain <a> tag.
  • Sections.jsx: Pass href={card.href} alongside to={card.to} in the documentation cards SmartLink to wire through the new prop.
// Before — triggers SPA router, 404s on static file
{ to: '/llms.txt', title: 'LLM-Native Docs', ... }

// After — plain <a href="..."> bypasses router
{ href: '/llms.txt', title: 'LLM-Native Docs', ... }

SmartLink already handles this: to → Docusaurus Link (SPA), no to + href<a> (direct navigation).

@netlify

netlify Bot commented Jun 15, 2026

Copy link
Copy Markdown

Deploy Preview for lucky-concha-f3599f ready!

Name Link
🔨 Latest commit bab866e
🔍 Latest deploy log https://app.netlify.com/projects/lucky-concha-f3599f/deploys/6a2ff86446fb900008ab54f6
😎 Deploy Preview https://deploy-preview-415--lucky-concha-f3599f.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.

Copilot AI changed the title [WIP] Fix traderX portal LLM link navigation issue fix: bypass SPA routing for /llms.txt link Jun 15, 2026
Copilot AI requested a review from DovOps June 15, 2026 13:05
@DovOps DovOps marked this pull request as ready for review June 15, 2026 14:09
@DovOps DovOps merged commit 26db8c9 into main Jun 15, 2026
11 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.

3 participants