-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cannot add reviewers to existing review on GitHub #225
Cannot add reviewers to existing review on GitHub #225
Comments
@thermopyle I wonder if you managed to find the cause/fix by any chance? I'm seeing this as well. |
Do you guys mean the prompt is empty and has no suggestions even if you type something in it? If so then the same here. Just found out I can add reviewers in forge via forge-edit-topic-review-request but it would be better to do that in code-review instead. |
Yeah that's it. I think I found the culprit: I believe the problem is how the cursor is used in the GraphQL query (Or Github implementation of it?). Basically for the initial query cursor is
|
On first query the cursor must be set to null. We set null by default and don't provide any cursor value on first query Fixes wandersoncferreira#225
Describe the bug
If I select
'r s r'
in an open GitHub review to request reviewers for the review, I cannot input anything at the prompt"Request review:"
. No matter what I input (email adress, GitHub username, with or without @ prefix), requesting a review fails with the error message"string-trim-right: Wrong type argument: stringp, nil"
. Pressing<TAB>
in the input field also does not work, the reply is[No matches]
.To Reproduce
Steps to reproduce the behavior:
'r s r'
to request reviewers.<TAB>
to select from list.Expected behavior
If I input a username or email adress, or if I select from a list returned by pressing
<TAB>
, the specified GitHub user should be requested to review.Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: