Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chore: revert open in coder docs for now #4154

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

Merged
merged 2 commits into from
Sep 22, 2022
Merged

chore: revert open in coder docs for now #4154

merged 2 commits into from
Sep 22, 2022

Conversation

bpmct
Copy link
Member

@bpmct bpmct commented Sep 22, 2022

We decided we want to add additional functionality + a new button before we document this feature.

also there were a few typos I missed

Screen Shot 2022-09-22 at 11 28 12 AM

@bpmct bpmct requested a review from jsjoeio September 22, 2022 16:31
Copy link
Contributor

@jsjoeio jsjoeio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want to revert the changes to dogfood/README.md as well?

bpmct referenced this pull request Sep 22, 2022
* docs: add open in coder

This adds new documentation for the "Open in Coder" button that admins
can use to get their developers up and running faster.

* fix: display error if template not found

Previously, we weren't handling a case where we tried to get a template
that returned a 404 from the backend.

Now we handle that case in our state machine and display the error
message from the API on the frontend.

* feat: support template query param in index

This adds support to navigate directly to a template from the index by
using the `?template=<name>` query  param.

* Revert "feat: support template query param in index"

This reverts commit bad7ffb.

We decided to use the `/template/path` route instead.

* fixup!: docs: add open in coder

* docs: add open in coder to dogfood readme

* Update docs/admin/open-in-coder.md

Co-authored-by: Ben Potter <[email protected]>

* Update docs/admin/open-in-coder.md

Co-authored-by: Ben Potter <[email protected]>

* Update docs/admin/open-in-coder.md

Co-authored-by: Ben Potter <[email protected]>
@bpmct
Copy link
Member Author

bpmct commented Sep 22, 2022

Do you want to revert the changes to dogfood/README.md as well?

I got it in this PR

@bpmct bpmct enabled auto-merge (squash) September 22, 2022 17:18
@bpmct bpmct merged commit 9e099b5 into main Sep 22, 2022
@bpmct bpmct deleted the bpmct/revert-oic branch September 22, 2022 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants