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)
Sat, Mar 23, 12:43 PM
Unknown Object (File)
Mon, Mar 18, 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
Unknown Object (File)
Jan 15 2024, 4:44 PM
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.