@chad - I'm not sure, but I think this is a different bug. T8080 is if the diff is accepted, which works just fine for me. This bug (T9311) is if the diff is requires changes

Sep 2 2015, 2:54 PM · Differential, Restricted Project

Sep 1 2015

mjlyons created T9311: Marking a diff as Requires Changes no longer removes it from Blocking Others.

Sep 1 2015, 6:46 PM · Differential, Restricted Project

Aug 14 2015

mjlyons added a comment to T9181: Add a "Ready for Review" state to diffs.

@epriestley - Whoa, that's awesome... sorry to bother you with this!

Aug 14 2015, 3:18 PM · Differential, Restricted Project

mjlyons added a comment to T9181: Add a "Ready for Review" state to diffs.

@epriestley - hmm, good points. I think it's mostly due to wanting to run tests. I'll monitor this and let you know if there's any other significant sources.

Aug 14 2015, 3:14 PM · Differential, Restricted Project

mjlyons created T9182: Require all Differential reviews to accept before greenlighting a review.

Aug 14 2015, 2:49 PM · Differential, Restricted Project

mjlyons created T9181: Add a "Ready for Review" state to diffs.

Aug 14 2015, 2:43 PM · Differential, Restricted Project