Differential D10087

Maybe fix excluding to / cc folks on differential emails
ClosedPublic
Actions

Authored by btrahan on Jul 30 2014, 11:06 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 1, 8:47 AM

	Unknown Object (File)
	Sun, Sep 1, 5:53 AM

	Unknown Object (File)
	Sat, Aug 31, 12:16 PM

	Unknown Object (File)
	Tue, Aug 27, 8:49 AM

	Unknown Object (File)
	Sun, Aug 25, 1:01 PM

	Unknown Object (File)
	Sun, Aug 25, 12:04 PM

	Unknown Object (File)
	Sun, Aug 25, 11:41 AM

	Unknown Object (File)
	Sat, Aug 24, 9:04 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T5185: Comment emails generated from parsing incoming email replies should not go to people already on the To/CC of the incoming email

Commits

Restricted Diffusion Commit
rPb2116a88636a: 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

Diff Detail

Repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan updated this revision to Diff 24255.Jul 30 2014, 11:06 PM

btrahan retitled this revision from to Maybe fix excluding to / cc folks on differential emails.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

btrahan added a task: T5185: Comment emails generated from parsing incoming email replies should not go to people already on the To/CC of the incoming email.

Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJul 30 2014, 11:06 PM

Harbormaster completed remote builds in B1947: Diff 24255.Jul 30 2014, 11:07 PM

epriestley accepted this revision.Jul 30 2014, 11:14 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jul 30 2014, 11:14 PM

Closed by commit rPb2116a88636a (authored by @btrahan).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

mail/

DifferentialReplyHandler.php

14 lines

Diff 24263

src/applications/differential/mail/DifferentialReplyHandler.php

Loading...