-
Notifications
You must be signed in to change notification settings - Fork 889
add docs for web IDEs (code-server, JetBrains Projector, VNC) #2448
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
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.
Amazing!
docs/manifest.json
Outdated
} | ||
] | ||
{ | ||
"title": "Configuring Web IDEs", |
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.
There's tension in my mind on this title because the app feature can be used for so much more than just configuring IDEs. But, I see how this is clearer for the average user.
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.
Yeah, for advanced usage, the Terraform provider docs is probably sufficient. We can mention it in a "building templates" page
docs/manifest.json
Outdated
{ | ||
"title": "Configuring Web IDEs", | ||
"description": "Learn how to configure web IDEs in your templates", | ||
"path": "./templates/configuring-web-ides.md" |
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.
Similarly to how I have Dotfiles as a top-level item, I think this configuration section should be listed under the root IDEs section. I think it's more natural that a user would want to see everything related to IDEs in Coder in one place.
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.
Will do
Resolves #2324 #2326 #2362
related: #2243