Homec4science

Maybe fix excluding to / cc folks on differential emails

Authored by Bob Trahan <btrahan@phacility.com> on Jul 31 2014, 03:50.

Description

Maybe fix excluding to / cc folks on differential emails

Summary: Ref T5185. By code inspection, I am pretty sure before this patch it was doing a set of a get on itself which does nothing. Now, being careful not to break Facebook we get the proper exclusion phids. I am pretty sure the folks in T5185 are experiencing this in Differential only.

Test Plan: Get some folks on T5185 to play with this

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T5185

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

Details

Committed
Bob Trahan <btrahan@phacility.com>Jul 31 2014, 03:50
Pushed
aubortJan 31 2017, 17:16
Parents
rPHcad41ea2947e: Implement build simulation; convert Harbormaster to be purely dependency based
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPHb2116a88636a: Maybe fix excluding to / cc folks on differential emails (authored by Bob Trahan <btrahan@phacility.com>).Jul 31 2014, 03:50