Page MenuHomePhabricator

When the last rejecting reviewer resigns from a revision, return it to "Needs Review"
ClosedPublic

Authored by epriestley on Mon, Nov 26, 2:06 PM.

Details

Summary

Ref T13216. Fixes T12920. See PHI911. If you reject a revision and then resign from it, it stays in "Needs Revision".

There's some arguable motivation for this, but it's inconsistent with how "Accept" works (if the last accepting reviewer resigns, we kick you out of "Accepted"). Make it consistent.

Test Plan
  • As the only reviewer: requested changes to a revision, then resigned.
  • Before: revision stays in "Needs Revision".
  • After: revision moves back to "Needs Review".

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Mon, Nov 26, 2:06 PM
epriestley requested review of this revision.Mon, Nov 26, 2:07 PM
amckinley accepted this revision.Mon, Nov 26, 4:32 PM
This revision is now accepted and ready to land.Mon, Nov 26, 4:32 PM
This revision was automatically updated to reflect the committed changes.