Skip to content

Fix limited user cannot view himself's profile#21212

Merged
wxiaoguang merged 3 commits into
go-gitea:release/v1.17from
lunny:lunny/fix_limited_user_view_self2
Sep 20, 2022
Merged

Fix limited user cannot view himself's profile#21212
wxiaoguang merged 3 commits into
go-gitea:release/v1.17from
lunny:lunny/fix_limited_user_view_self2

Conversation

@lunny

@lunny lunny commented Sep 19, 2022

Copy link
Copy Markdown
Member

backport #21210

@lunny lunny added the type/bug label Sep 19, 2022
@lunny lunny added this to the 1.17.3 milestone Sep 19, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Sep 19, 2022
@delvh

delvh commented Sep 19, 2022

Copy link
Copy Markdown
Member

Should we fix the incorrect line 1307 as well here?

@lunny

lunny commented Sep 20, 2022

Copy link
Copy Markdown
Member Author

Should we fix the incorrect line 1307 as well here?

Done.

@lunny

lunny commented Sep 20, 2022

Copy link
Copy Markdown
Member Author

Missing tests.

@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 Sep 20, 2022
@wxiaoguang wxiaoguang merged commit f663773 into go-gitea:release/v1.17 Sep 20, 2022
@lunny lunny deleted the lunny/fix_limited_user_view_self2 branch September 20, 2022 08:49
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
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