Page MenuHomePhabricator

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

Authored by epriestley on Feb 20 2020, 8:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 18, 4:04 AM
Unknown Object (File)
Thu, Sep 8, 11:38 PM
Unknown Object (File)
Wed, Sep 7, 10:22 AM
Unknown Object (File)
Wed, Sep 7, 9:40 AM
Unknown Object (File)
Mon, Sep 5, 8:25 AM
Unknown Object (File)
Mon, Sep 5, 6:17 AM
Unknown Object (File)
Sat, Sep 3, 6:41 AM
Unknown Object (File)
Sat, Sep 3, 5:03 AM
Subscribers
None

Details

Summary

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

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
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.