Suggestion: Rename "Reviews" on new sidebar with "Reviewers with write access" #169946
Replies: 2 comments
-
💬 Your Product Feedback Has Been Submitted 🎉 Thank you for taking the time to share your insights with us! Your feedback is invaluable as we build a better GitHub experience for all our users. Here's what you can expect moving forward ⏩
Where to look to see what's shipping 👀
What you can do in the meantime 💻
As a member of the GitHub community, your participation is essential. While we can't promise that every suggestion will be implemented, we want to emphasize that your feedback is instrumental in guiding our decisions and priorities. Thank you once again for your contribution to making GitHub even better! We're grateful for your ongoing support and collaboration in shaping the future of our platform. ⭐ |
Beta Was this translation helpful? Give feedback.
-
Hello! Thanks for your feedback, and completely understand. Unfortunately that won't always be true of that section - for example, it can also contain pending reviews regardless of write access. There are such a wide variety of ways that various projects handle reviewers and what it means, and we can definitely continue to take feedback on how to be more flexible for these scenarios. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Select Topic Area
Product Feedback
Body
Today I noticed that the reviewer panel on pull requests now distinguishes between potentially blocking reviews and all other reviews. Which seems like it could be a good change, except that it's not immediately clear what distinction is being made between the two types of reviews (unless you read and understand the tooltip), and can appear to delegitimize reviews made by project members without write access such as dedicated reviewers with triage perms.
I would much appreciate it if the title at the top of the panel in this new sidebar design was changed to be more descriptive (something along the lines of "Reviewers with write access" would work well).
Beta Was this translation helpful? Give feedback.
All reactions