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

Skip to content

fix: rename group GET request #9097

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
Aug 15, 2023
Merged

fix: rename group GET request #9097

merged 2 commits into from
Aug 15, 2023

Conversation

ericpaulsen
Copy link
Member

this PR fixes the naming & description of the /groups/{group} route. the endpoint is incorrectly described as "Get group by name", whereas the API call expects the group UUID to be passed in.

@mtojek this is my first PR against the API code. please let me know if I missed anything.

@ericpaulsen ericpaulsen added docs Area: coder.com/docs api Area: HTTP API bug labels Aug 15, 2023
@ericpaulsen ericpaulsen requested review from mtojek and bpmct August 15, 2023 01:14
@ericpaulsen ericpaulsen self-assigned this Aug 15, 2023
@ericpaulsen ericpaulsen changed the title fix: group GET req naming fix(api): rename group GET request Aug 15, 2023
@ericpaulsen ericpaulsen changed the title fix(api): rename group GET request fix: rename group GET request Aug 15, 2023
@bpmct bpmct removed their request for review August 15, 2023 14:38
@ericpaulsen ericpaulsen merged commit e4c24e0 into main Aug 15, 2023
@ericpaulsen ericpaulsen deleted the fix-get-group-api branch August 15, 2023 18:47
@github-actions github-actions bot locked and limited conversation to collaborators Aug 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api Area: HTTP API docs Area: coder.com/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants