Skip to content
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

API Endpoint to add a member to an organisation #3462

Closed
wants to merge 2 commits into from

Conversation

culmat
Copy link

@culmat culmat commented Feb 5, 2018

completing API Endpoints for organization members #645

@lafriks
Copy link
Member

lafriks commented Feb 5, 2018

Thanks for PR but there is no real need in Gitea for this method as we have already removed from interface as adding user to org members without adding to team does not give anything (including rights to see any private repos etc) as Gitea has a bit different rights model

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 5, 2018
@lafriks lafriks added the status/blocked This PR cannot be merged yet, i.e. because it depends on another unmerged PR label Feb 5, 2018
@culmat
Copy link
Author

culmat commented Feb 6, 2018

I see now.
PUT /teams/{id}/members/{username} Add a team member
also adds to the org. I was mislead by the error message "Must be an organization member". It occurred when listing the teams on /orgs/{orgName}/teams.
Closing this.

@culmat culmat closed this Feb 6, 2018
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. status/blocked This PR cannot be merged yet, i.e. because it depends on another unmerged PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants