-
Notifications
You must be signed in to change notification settings - Fork 899
docs: separate coder desktop sections into install and use pages #18068
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.
Left a few suggestions. This looks great 😃
|
||
If you encounter a synchronization conflict, delete the conflicting file that contains changes you don't want to keep. | ||
|
||
## Accessing web apps in a secure browser context |
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.
Can we move this to a troubleshooting section. This is not something we should highlight.
Also if we can use a compacted view for troubleshooting topics to prevent them from occupying a significant space on page
|
||
## Troubleshooting | ||
|
||
### Mac: Issues updating Coder Desktop |
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.
@ethanndickson can we remove this section now? Or replace with the helper service screenshot and instructions?
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 we can get rid of this whole Issues updating Coder Desktop
section
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.
Thanks @EdwardAngert. Please move all suggestions to the new issue so we do not miss them.
preview
we should consider the "Connect" naming in future PRs