-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Fix: Add missing /customize redirect to overview page #8692
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
β¦stomize/ This consolidation addresses CON-4476 by eliminating confusing duplicate documentation paths. ## Changes Made ### Structure Consolidation - Removed /docs/customization/ directory entirely - Kept /docs/customize/ as the single canonical path (it contains comprehensive content including deep-dives, model-providers, and model-roles) - Moved better overview content from /customization/overview.mdx to /customize/overview.mdx - Added new top-level pages to /customize/: models.mdx, mcp-tools.mdx, rules.mdx, prompts.mdx, settings.mdx ### Navigation Updates - Removed duplicate "Customization" tab from docs.json navigation - Updated "Customize" tab to include the new top-level pages - All customization content now appears in a single, well-organized tab ### Redirects - Added redirects from all /customization/* paths to /customize/* - Ensures backward compatibility for bookmarks and external links - Updated existing redirects that pointed to /customization/overview ### Link Updates - Updated 16+ internal documentation links from /customization/ to /customize/ - Updated image paths from /images/customization/ to /images/customize/ - Consolidated image directories (removed /docs/images/customization/) ## Reasoning for Content Decisions ### Why Keep /customize/ Over /customization/? 1. More concise - Shorter path is easier to type and remember 2. Complete structure - Already contained extensive subdirectories (deep-dives/, model-providers/, model-roles/) 3. Active development - Model provider docs and deep-dives are actively maintained here 4. Better organization - Clear hierarchy with specialized subdirectories ### Content Consolidation Strategy - Overview page: Used /customization/overview.mdx as base (clearer introduction and practical examples) - New pages: Brought over /customization/ files that provided good card-based overviews for models, rules, prompts, MCP, and settings - Deep-dives: Kept existing comprehensive /customize/deep-dives/ content (no overlap with /customization/) - Model providers: Kept existing /customize/model-providers/ (no equivalent in /customization/) - Model roles: Kept existing /customize/model-roles/ (no equivalent in /customization/) ### Deleted Content Justification All files from /docs/customization/ were either: 1. Duplicates with less comprehensive content than /customize/ equivalents 2. Moved to /customize/ to serve as helpful overview pages 3. No unique information lost - content was either redundant or merged ## Success Metrics Addressed - Zero duplicate documentation paths - All internal links functional (verified) - Clear single source of truth for customization - Backward compatibility via redirects Generated with [Continue](https://continue.dev) Co-Authored-By: Continue <[email protected]> Co-authored-by: Username <[email protected]>
- Remove conflicting redirect from /customize/settings to /customize/overview - Restore 'Change Your Model Provider' section to overview page - Restore introductory copy explaining customization options - Add redirect for /customization/overview#change-your-model-provider anchor Addresses feedback from #8469 (review) Co-authored-by: bdougieyo <[email protected]> Generated with [Continue](https://continue.dev) Co-Authored-By: Continue <[email protected]>
- Fixes issue where /customize path had no redirect - Ensures users navigating to /customize are directed to the overview page - Addresses feedback from PR review #8469 Co-authored-by: bdougieyo <[email protected]>
|
There was a problem hiding this 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 29 files
|
Created this agent in error. |
Summary
This PR addresses feedback from PR #8469 review by adding a missing redirect for the
/customizepath.Changes Made
π Redirect Addition
/customizeβ/customize/overview/customizewithout a specific page are directed to the overviewProblem Addressed
From the PR review feedback:
While
/customization/*paths were properly redirected to/customize/*, there was no redirect for the bare/customizepath itself. This could cause 404 errors if users or systems try to access/customizedirectly.Testing
β Redirect configured -
/customizenow redirects to/customize/overviewβ Existing redirects intact - All
/customization/*paths still workβ Navigation consistency - Aligns with other base path redirects (e.g.,
/customizationβ/customize/overview)Additional Notes
The anchor link issue mentioned in the review (
/customization/overview#change-your-model-provider) already has a proper redirect configured and the anchor exists in the target file, so no additional changes were needed for that.This agent session was co-authored by bdougieyo and Continue.
Summary by cubic
Adds a missing redirect for /customize to /customize/overview and consolidates duplicate docs paths to make /customize the single canonical section. Addresses Linear issue CON-4476 and resolves dead-link feedback from PR #8469.
Bug Fixes
Refactors
Written for commit 32fe915. Summary will update automatically on new commits.