HomePhorge

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

Description

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

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".

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10967

Differential Revision: https://secure.phabricator.com/D17568

Details

Provenance
epriestleyAuthored on Mar 28 2017, 6:22 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP415ad784844e: Remove old code for "Request Review" action from Differential
Branches
Unknown
Tags
Unknown

Event Timeline