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.

Sign up for GitHub

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

Jump to bottom

fix admin lost permission caused by #947 #1753

Merged
merged 1 commit into from
May 19, 2017
Merged

fix admin lost permission caused by #947 #1753

merged 1 commit into from
May 19, 2017

Conversation

lunny
Copy link
Member

@lunny lunny commented May 19, 2017

This will fix #1751.

@lunny lunny added the type/bug label May 19, 2017
@lunny lunny added this to the 1.2.0 milestone May 19, 2017
@lunny lunny mentioned this pull request May 19, 2017
Closed
7 tasks
@lafriks
Copy link
Member

lafriks commented May 19, 2017

LGTM

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

DblK commented May 19, 2017

LGTM

@lafriks
Copy link
Member

lafriks commented May 19, 2017

Make LG-TM work

@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 May 19, 2017
@bkcsoft bkcsoft merged commit 281a038 into go-gitea:master May 19, 2017
@wyattoday wyattoday mentioned this pull request May 19, 2017
Closed
7 tasks
@lunny lunny deleted the lunny/fix_admin_perm branch October 26, 2017 15:22
@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.
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. type/bug
Projects
None yet
Milestone
1.2.0
Development

Successfully merging this pull request may close these issues.

Pull request #947 breaks HTTPS push/pull to "master" when logged in as Admin
5 participants
@lunny @lafriks @DblK @tboerger @bkcsoft