Review Comment API missing side/start-line #86055
Unanswered
zimventures
asked this question in
Apps, API and Webhooks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Select Topic Area
Bug
Body
When making a call to list the comments for a pull review, I don't appear to be seeing any of the details for the side or start/end lines for a multi-line comment.
For example - here is a review comment made on the right side of a PR:

When performing the API call to get the comments for the review, all I get back is:
[ { "id": 1446295484, "url": "[redacted]", "pull_request_review_id": 1811492167, "diff_hunk": "@@ -9,6 +9,10 @@\n \n This module contains all of the code to execute Google Cloud Functions. Each subdirectory contains a single function that will be deployed within the GCloud. During local development, each function will run as a standalone Flask application. The Flask applications will all run on different ports.\n \n+# This is nothing\n+ignore this text\n+Just looking to do a multi-line change\n+", "path": "functions/README.md", "position": 7, "original_position": 7, "commit_id": "a22506f11a44a3baa09f11874d170676d97d2a08", "user": { [redacted] }, "body": "This is a multi-line comment on the right side to talk about something.", "created_at": "2024-01-09T16:08:42Z", "updated_at": "2024-01-09T16:09:00Z", "html_url": "[redacted]", "pull_request_url": "[redacted]", "author_association": "CONTRIBUTOR", "_links": { [redacted] }, "original_commit_id": "a22506f11a44a3baa09f11874d170676d97d2a08", "reactions": { [redacted] } } ]Even though the comment spans 3 lines (12-15), the only offset information received is a single relative location in the diff hunk - excluding any notion that the comment was made on multiple lines.
I feel like this is a bug because the API docs mention the presence of
side,start_side,line,original_line,start_line, andoriginal_start_line, none of which appear to be present in the results from the API call.Beta Was this translation helpful? Give feedback.
All reactions