HomePhorge

Don't include resigned reviewers in the Differential "To" list

Description

Don't include resigned reviewers in the Differential "To" list

Summary: Ref T12689. See PHI178. This isn't a complete solution (you may still get mailed via packages/projects) but should fix the obvious issue, where "Resigned" reviewers are incorrectly always sent mail directly.

Test Plan: Had Alice resign, interacted as Bailey, no mail to Alice.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T12689

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

Details

Provenance
epriestleyAuthored on Nov 1 2017, 11:08 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP6ecdadb76a95: After "Request Review", move revisions with voided "Accepts" into "Ready to…
Branches
Unknown
Tags
Unknown

Event Timeline