Skip to content

Fix stale Models docs links in config panel#11136

Open
siewcapital wants to merge 2 commits intocontinuedev:mainfrom
Siew-s-Capital:fix/models-learn-more-links-11126
Open

Fix stale Models docs links in config panel#11136
siewcapital wants to merge 2 commits intocontinuedev:mainfrom
Siew-s-Capital:fix/models-learn-more-links-11126

Conversation

@siewcapital
Copy link

@siewcapital siewcapital commented Mar 6, 2026

Summary

  • update the Models section learn-more links for Chat, Autocomplete, and Edit to the current ide-extensions/.../quick-start docs routes
  • update model setup links in the same section to ide-extensions/.../model-setup
  • add a focused regression test to verify all six links point at the expected routes and do not contain /docs/

Why

The links in the Models panel were still pointing at older paths. This aligns the UI with the current docs structure and prevents regressions.

Closes #11126.

Validation

  • npm test -- src/pages/config/sections/ModelsSection.test.tsx (from gui/) → fails in this local env due to unresolved transitive module imports from core test graph (partial-json)
  • git diff --check

Summary by cubic

Fixes stale docs links in the Models config panel by switching all "Learn more" and model setup URLs to the current ide-extensions/.../quick-start and ide-extensions/.../model-setup routes, centralizing them in MODEL_DOCS_URLS, and removing a redundant test to avoid duplicate truth. Closes #11126.

Written for commit 3e44346. Summary will update on new commits.

@siewcapital siewcapital requested a review from a team as a code owner March 6, 2026 19:08
@siewcapital siewcapital requested review from RomneyDa and removed request for a team March 6, 2026 19:08
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 6, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Copy link
Collaborator

@RomneyDa RomneyDa left a comment

Choose a reason for hiding this comment

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

@siewcapital thanks for the link fixes!

Could you update the tests to use imported MODEL_DOCS_URLS or similar to prevent 2 sources of truth? Or feel free to remove the test file in this case.

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Mar 10, 2026
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Mar 14, 2026
@siewcapital
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@siewcapital
Copy link
Author

@RomneyDa Makes total sense to avoid duplicating the source of truth. I've gone ahead and just dropped the test file entirely as you suggested, which keeps things clean and simple.

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

Labels

ai-merge size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Models panel "Learn more" links point to outdated /docs documentation paths

2 participants