Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

API Endpoints for organization members #645

Merged
merged 1 commit into from
Jan 20, 2017
Merged

API Endpoints for organization members #645

merged 1 commit into from
Jan 20, 2017

Conversation

ethantkoenig
Copy link
Member

@ethantkoenig ethantkoenig commented Jan 11, 2017 •
edited
Loading

Adds the following endpoints:

  • GET /orgs/:org/members: List an organization's members
  • GET /orgs/:org/members/:user: Check if a user is a member of an organization
  • DELETE /orgs/:org/members/:user: Remove a user from an organization
  • GET /orgs/:org/public_members: List an organization's public members
  • GET /orgs/:org/public_member/:user: Check if a user is a public member of an organization
  • PUT /orgs/:org/public_member/:user: Make a user's membership public
  • DELETE /orgs/:org/public_member/:user: Make a user's membership non-public

Also fix a bug in GetUsersByIDs.

@lunny lunny added this to the 1.1.0 milestone Jan 11, 2017
@lunny lunny added modifies/api This PR adds API routes or modifies them type/feature Completely new functionality. Can only be merged if feature freeze is not active. labels Jan 11, 2017
@ethantkoenig ethantkoenig force-pushed the api/org_members branch 2 times, most recently from 42f7c7b to c99ef1a Compare January 14, 2017 00:27
@lunny
Copy link
Member

lunny commented Jan 14, 2017

LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 14, 2017
@Bwko
Copy link
Member

Bwko commented Jan 19, 2017

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jan 19, 2017
@lunny lunny merged commit fcf02e4 into go-gitea:master Jan 20, 2017
@ethantkoenig ethantkoenig deleted the api/org_members branch January 22, 2017 17:20
@culmat culmat mentioned this pull request Feb 5, 2018
Closed
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers
No reviews
Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/api This PR adds API routes or modifies them type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Projects
None yet
Milestone
1.1.0
Development

Successfully merging this pull request may close these issues.
4 participants
@ethantkoenig @lunny @Bwko @tboerger