Homec4science

Clean up Maniphest email "to"

Authored by epriestley <git@epriestley.com> on Sep 25 2013, 23:37.

Description

Clean up Maniphest email "to"

Summary:
We currently try to send Maniphest email "To" the owner and actor, but for unassigned tasks there is no owner.

Just filter the PHIDs in the parent, since it's reasonable for subclasses to be liberal about construction here.

Test Plan: Commented on an unassigned task, got an email without a bogus "To".

Reviewers: btrahan, asherkin

Reviewed By: btrahan

CC: aran

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

Details

Committed
epriestley <git@epriestley.com>Sep 25 2013, 23:37
Pushed
aubortJan 31 2017, 17:16
Parents
rPH1e2718d747e8: Make Maniphest list page react to viewer capabilities
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHb8154cb5e91a: Clean up Maniphest email "to" (authored by epriestley <git@epriestley.com>).Sep 25 2013, 23:37