Page MenuHomePhabricator

Make updates of rejected revisions behave correctly again
ClosedPublic

Authored by epriestley on Mar 5 2014, 1:55 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 4, 12:47 PM
Unknown Object (File)
Tue, Dec 3, 8:39 PM
Unknown Object (File)
Tue, Dec 3, 11:23 AM
Unknown Object (File)
Nov 27 2024, 12:26 AM
Unknown Object (File)
Nov 27 2024, 12:26 AM
Unknown Object (File)
Nov 27 2024, 12:26 AM
Unknown Object (File)
Nov 24 2024, 9:45 PM
Unknown Object (File)
Nov 24 2024, 5:47 PM
Subscribers

Details

Summary

Ref T2222. Ref T4481. Specifically:

  • When a revision is updated, change all "Reject" reviewers to "Reject Prior".
  • Change status to "Needs Review".
  • Update the state logic to account for this properly.
Test Plan
  • Created a revision as user A, with B as a reviewer.
  • Rejected as B.
  • Updated the revision as A.
  • Saw revision in "needs review" state, with B as a "Rejected Prior" reviewer.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 5 2014, 6:35 PM