Homestyx hydra
Diffusion hydra 1e8ed3e5ffc7

Treat "request review" more like an update

Description

Treat "request review" more like an update

Summary:
After "reject; plan changes; request review", revisions go back to "needs revision". Instead, they should remain in "needs review" (the reviewers need to review comments on the "request review", in the normal case). Generally, "request reivew" should act a lot like "update", just not actually change the diff.

To accomplish this, downgrade reviewers on "request review" to "rejected older", just like we would on an update.

Test Plan: Did "reject; plan; request", revision ended in "needs review". Rejected it into "needs revision"; updated it into "needs review".

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: dctrwatson, epriestley

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

Details

Provenance
epriestleyAuthored on Mar 17 2014, 9:19 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:5b2887b69b0e: Add "Date Updated" query fields for Maniphest
Branches
Unknown
Tags
Unknown

Event Timeline