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 /admin/users/{username} missing parameter #4774

Closed
wants to merge 4 commits into from

Conversation

EnricoFerro
Copy link
Contributor

@EnricoFerro EnricoFerro commented Aug 23, 2018

This PR modify the logic of the api /admin/users/{username} as descripted in the issue #4765

Add two field in the structure EditUserOption and allow to modify the values for Disable Sign-In and
May Create Organization
image

The new two field in the swagger editor
image

@EnricoFerro EnricoFerro changed the title Fix for issue #4765 API /admin/users/{username} missing parameter Aug 23, 2018
@daviian
Copy link
Member

daviian commented Aug 23, 2018

@EnricoFerro Please don't make changes directly to the vendor folder. Instead create another PR for that (referring to https://github.com/go-gitea/go-sdk)

@bkcsoft bkcsoft added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 23, 2018
@EnricoFerro
Copy link
Contributor Author

My fault.. ..close this PR and recreate ( after go-gitea/go-sdk#114 ) approvation

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants