Skip to content

Conversation

@ethantkoenig
Copy link
Member

Fixes #1895

Copy link
Contributor

Choose a reason for hiding this comment

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

var requestingID int64

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated

@andreynering
Copy link
Contributor

LGTM

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

Bwko commented Jun 7, 2017

LGTM

@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 Jun 7, 2017
@lunny lunny added this to the 1.2.0 milestone Jun 8, 2017
@lunny
Copy link
Member

lunny commented Jun 8, 2017

It seems CI failed. @ethantkoenig

@ethantkoenig ethantkoenig force-pushed the fix/public_activity branch from 89daccb to 836ae95 Compare June 8, 2017 02:14
@ethantkoenig
Copy link
Member Author

Rebased, since CI did not run for some reason

@appleboy appleboy merged commit afe1f1c into go-gitea:master Jun 8, 2017
@ethantkoenig ethantkoenig deleted the fix/public_activity branch June 8, 2017 14:23
@lunny lunny added the type/bug label Aug 25, 2017
@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/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.

500 on user's public activity page

6 participants