1240955 - It's not obvious how to r+ a patch

Status: RESOLVED INVALID

(MozReview Graveyard :: General, defect)

Description

[François Marier [:francois]]

I just wanted to give some feedback on the review experience in case that's useful.

Giving someone an r+ after reviewing a patch is kinda confusing:

1. click "Finish Review"
2. click "Ship It"
3. click "Ok" (is this even needed?)
4. realize there's a publish button at the bottom right of the page and click "Publish Review"
5. do all of this again for the other patch in the same patch series

(That might be why gcp prefers I submit Safe Browsing patches for review outside of MozReview.)

It would be nice to just be able to click the [r?] in the Status column and change it to a [r+] there.

Comment 1

[BMO Automation]

MozReview is now obsolete. Please use Phabricator instead. Closing this bug.