Skip to content

Conversation

lunny
Copy link
Member

@lunny lunny commented Jan 20, 2020

partially backport #9842 and fix more wrong issue permission check.

Hint: If a page or an API is shared by issue and pullrequest, you must use CanWriteIssuesOrPulls(issue.IsPull) but not CanWrite(models.UnitTypeIssues) to check the permisson.

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 20, 2020
Copy link
Member

@guillep2k guillep2k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one nit, otherwise LGTM

@GiteaBot GiteaBot 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 20, 2020
@lunny lunny force-pushed the lunny/fix_dependcies3 branch from 69f5290 to fe27e0b Compare January 20, 2020 15:05
@sapk sapk merged commit c4e0f71 into go-gitea:release/v1.10 Jan 20, 2020
@lunny lunny deleted the lunny/fix_dependcies3 branch January 20, 2020 15:50
zeripath added a commit to zeripath/gitea that referenced this pull request Feb 16, 2020
lafriks pushed a commit that referenced this pull request Feb 16, 2020
* v1.10.4 Changelog

* Add backport identifier for #10261

* Update CHANGELOG.md entry for #9884

* Workaround docker-library/postgres#658

* Update .drone.yml
zeripath added a commit to zeripath/gitea that referenced this pull request Feb 16, 2020
* v1.10.4 Changelog

* Add backport identifier for go-gitea#10261

* Update CHANGELOG.md entry for go-gitea#9884
zeripath added a commit to zeripath/gitea that referenced this pull request Feb 16, 2020
* v1.10.4 Changelog

* Add backport identifier for go-gitea#10261

* Update CHANGELOG.md entry for go-gitea#9884
techknowlogick pushed a commit that referenced this pull request Feb 16, 2020
* v1.10.4 Changelog

* Add backport identifier for #10261

* Update CHANGELOG.md entry for #9884
techknowlogick pushed a commit that referenced this pull request Feb 16, 2020
* v1.10.4 Changelog

* Add backport identifier for #10261

* Update CHANGELOG.md entry for #9884

Co-authored-by: Antoine GIRARD <[email protected]>
@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/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants