- Notifications
You must be signed in to change notification settings - Fork927
docs: move api and cli docs routes to reference/#14241
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Do the routes need to be nested like that? The redirects feel unnecessary. |
@kylecarbs Aren't redirects required for any old links to work? in case someone is using those links in any internal docs?
I fixed the manifest. Routes are now nested under Reference. |
alwaysmeticulousbot commentedAug 13, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Meticulous spotted zero visual differences across 1354 screens tested:view results. Expected differences?Click here. Last updated for commit4f098d0. This comment will update as new commits are pushed. |
5978a06
toa0efc99
CompareI can make the redirect on coder.com whenever this is good to merge! |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
Nice work 👍 |
48f29a1
intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
As part of#13434, this PR moves CLI and API docs under reference.
This will help reduce the size of#14182.
we need to create redirects from docs/[cli,api] to docs/reference/[cli,api]
Docs link check is passing so it should be fine to merge.