Open
Bug 1393193
Opened 8 years ago
Updated 6 years ago
Warning but no Next Steps on revisions when the user has given no reviewers
Categories
(Conduit :: Phabricator, enhancement, P3)
Conduit
Phabricator
Tracking
(Not tracked)
NEW
People
(Reporter: mars, Unassigned)
Details
(Keywords: conduit-triaged)
Attachments
(1 file)
I sometimes create code reviews in Phabricator without first knowing who will be reviewing my code. When I do this I leave the list of reviewers on my revision empty, and I see a big yellow warning on the Revision page that says "This revision needs review, but there are no reviewers specified." However, I don't see a button that would take me straight to the "Add a reviewer" page to get rid of the warning.
To add a reviewer, I have to find and click "Edit Revision" in the right-hand menu, then find and click "Reviewers" on the edit page.
If adding a reviewer is an action important enough to put a Big Yellow Warning banner on the page, and if adding a reviewer is almost always the next step that I am going to take in the code review lifecycle, then it would be helpful to have a button that lets me take the "Add a reviewer" action right away without any extra clicks so I can clear the warning.
Two possible locations I would expect to see the button to add/edit reviewers would be in the warning banner itself or possibly in the right-hand actions menu of the Revision.
Comment 1•8 years ago
|
||
Comment 2•8 years ago
|
||
Comment on attachment 8940765 [details]
Install arc in its own container for development
David Walsh :davidwalsh has approved the revision.
Piotr Zalewa [:zalun] has approved the revision.
https://phabricator.services.mozilla.com/D2
Attachment #8940765 -
Flags: review+
Comment 3•7 years ago
|
||
I believe the revision here was associated with the wrong bug... throwing this into the backlog.
Keywords: conduit-triaged
Whiteboard: [phabricator-backlog]
Updated•7 years ago
|
Component: Phabricator Upstream → Phabricator
Keywords: conduit-backlog
Whiteboard: [phabricator-backlog]
Keywords: conduit-backlog
Priority: -- → P3
You need to log in
before you can comment on or make changes to this bug.
Description
•