Skip to content

Conversation

@sotho
Copy link
Contributor

@sotho sotho commented Mar 25, 2021

Backport of #15139

The API call: GET /repos/{owner}/{repo}/pulls/{index}/reviews/{id}/comments
returns always the reviewer, but should return the poster.

Co-authored-by: 6543 6543@obermui.de
Co-authored-by: zeripath art27@cantab.net

The API call: GET /repos/{owner}/{repo}/pulls/{index}/reviews/{id}/comments
returns always the reviewer, but should return the poster.

Co-authored-by: 6543 <6543@obermui.de>
Co-authored-by: zeripath <art27@cantab.net>
@6543 6543 added this to the 1.13.7 milestone Mar 25, 2021
@6543 6543 added modifies/api This PR adds API routes or modifies them type/bug labels Mar 25, 2021
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 25, 2021
@sotho
Copy link
Contributor Author

sotho commented Mar 25, 2021

@6543 I think we should re-work the fix, see #15139 (comment)

Copy link
Member

@6543 6543 left a comment

Choose a reason for hiding this comment

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

Blocking until #15139 (comment) got solved

@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 Mar 26, 2021
@lafriks lafriks merged commit 4f47bf5 into go-gitea:release/v1.13 Mar 26, 2021
@go-gitea go-gitea locked and limited conversation to collaborators May 13, 2021
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. modifies/api This PR adds API routes or modifies them type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants