Page MenuHomePhabricator

Remove old code for sending email to external users who create objects via inbound mail

Authored by epriestley on Feb 20 2020, 8:29 PM.
Referenced Files
Unknown Object (File)
Wed, Apr 3, 5:54 AM
Unknown Object (File)
Sat, Mar 23, 12:43 PM
Unknown Object (File)
Mar 18 2024, 2:43 PM
Unknown Object (File)
Feb 22 2024, 9:44 PM
Unknown Object (File)
Feb 21 2024, 1:07 PM
Unknown Object (File)
Feb 16 2024, 3:01 AM
Unknown Object (File)
Feb 3 2024, 11:41 PM
Unknown Object (File)
Jan 25 2024, 2:11 AM



Ref T13493. I'm updating callers to getAccountID() to prepare to move it to a separate table.

This callsite once supported this flow:

  • External users with no accounts send mail to bugs@.
  • This creates tasks in Maniphest.
  • They're CC'd when the tasks are updated.

However, after T12237 we never actually send this mail (since their addresses are necessarily unverified).

I left this code in in case this needed to be revisited, but it hasn't been an issue. Just remove it and treat these users as undeliverable.

Test Plan

As a cursory test for nothing being horribly broken, sent some object mail.

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 20 2020, 8:41 PM
This revision was automatically updated to reflect the committed changes.