- Notifications
You must be signed in to change notification settings - Fork913
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 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
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 😃
Uh oh!
There was an error while loading.Please reload this page.
| 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 wholeIssues 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.
e906ce2 intomainUh oh!
There was an error while loading.Please reload this page.
preview
we should consider the "Connect" naming in future PRs