wandersoncferreira / code-review

Code Reviews in Emacs
https://wandersoncferreira.github.io/code-review/
GNU General Public License v3.0
462 stars 49 forks source link

Reply comments lost when submitting with regular comments #204

Open mplanchard opened 2 years ago

mplanchard commented 2 years ago

Describe the bug When submitting a review (on GitHub) with both regular comments and reply comments, an error is thrown and the reply comments are lost (they do not show up on GitHub, and they disappear from the code-review interface).

The error is:

error in process filter: code-review-github-errback: Wrong type argument: characterp, (resource . "PullRequestReviewComment")
error in process filter: Wrong type argument: characterp, (resource . "PullRequestReviewComment")

To Reproduce

  1. Start a code review on which there are already some comments
  2. Add some comments with code-review-comment-add-or-edit
  3. Add some reply comments by calling code-review-comment-add-or-edit on existing comments
  4. Submit feedback (e.g. approve, request changes, comment)

Expected behavior Ideally both reply comments and regular comments should be submitted. If only regular comments are submitted, though, the reply comments should at least be kept to avoid losing work so that they can then be submitted individually with code-review-submit-only-replies

Screenshots N/A

Desktop (please complete the following information):

Additional context Add any other context about the problem here.