-
Notifications
You must be signed in to change notification settings - Fork 24
View Existing Comments on Code lines #5
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
Comments
That's a great suggestion and I was planning to work on that + the ability to respond to inline comments. I will be taking a look in the coming weeks. |
I started working on a solution (#10) @parhamdoustdar you can take a look and give feedback if interested. |
I will merge #10 after cleaning up the code and fixing bugs and implement comments on code lines in another PR. |
Hi @charignon - it looks like this feature is not working for me. I havethe feature flag set to |
Are you sure you are talking about top level comments? (that is comments on the PR body and not the code). |
here is the query to get the nested review comments and code:
But maybe we could use position or something to put the comment in the right place in the diff? I'm new to the graphql api. You'd probably need to build everything differently with |
Just as a notification, this feature is now in master behind a feature flag. Would be very nice to have user feedbacks, please set |
First of all, thank you a lot for this package. The Github code review interface sucks when using a screen reader, but this allows me to use it with Emacs and Emacspeak. Awesomeness!
One problem right now is that I don't see existing comments as I'm reviewing the code. It'd be great to see the existing comments as i'm reading.
The text was updated successfully, but these errors were encountered: