Page MenuHomePhabricator

When voiding "Accept" reviews, also void "Reject" reviews
ClosedPublic

Authored by epriestley on Mar 28 2017, 1:24 PM.
Tags
None
Referenced Files
F18828431: D17568.id42265.diff
Fri, Oct 24, 4:10 PM
F18827201: D17568.id.diff
Fri, Oct 24, 8:25 AM
F18821494: D17568.id.diff
Wed, Oct 22, 11:44 PM
F18818125: D17568.diff
Tue, Oct 21, 9:15 PM
F18769971: D17568.id42272.diff
Wed, Oct 8, 10:09 AM
F18769522: D17568.diff
Wed, Oct 8, 7:56 AM
F18698875: D17568.diff
Sep 27 2025, 11:59 AM
F18632332: D17568.id42272.diff
Sep 16 2025, 5:38 PM
Subscribers
None

Details

Summary

Ref T10967. This change is similar to D17566, but for rejects.

Test Plan
  • Create a revision as A, with reviewer B.
  • Reject as B.
  • Request review as A.
  • Before patch: stuck in "rejected".
  • After patch: transitions back to "needs review".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Mar 28 2017, 5:50 PM
This revision was automatically updated to reflect the committed changes.