Page MenuHomePhabricator

Update inbound and outbound email documentation
ClosedPublic

Authored by epriestley on Jan 14 2019, 11:21 PM.
Tags
None
Referenced Files
F14089398: D19973.diff
Sun, Nov 24, 10:09 AM
Unknown Object (File)
Oct 22 2024, 7:36 PM
Unknown Object (File)
Oct 15 2024, 5:06 AM
Unknown Object (File)
Oct 14 2024, 12:40 PM
Unknown Object (File)
Oct 9 2024, 11:12 AM
Unknown Object (File)
Oct 9 2024, 6:41 AM
Unknown Object (File)
Oct 5 2024, 8:30 AM
Unknown Object (File)
Oct 1 2024, 3:02 AM
Subscribers
None

Details

Summary

Fixes T8636. Mention Herald for inbound, update some outbound stuff, do some language / organization tweaks.

Test Plan

Read documentation.

Diff Detail

Repository
rP Phabricator
Branch
mfa17
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 21526
Build 29334: Run Core Tests
Build 29333: arc lint + arc unit

Event Timeline

  • Use consistent typography for "SendGrid".

Two actual typos; feel free to ignore the rest if you disagree with the language changes.

src/docs/user/configuration/configuring_inbound_email.diviner
18

This is a little less awkward to parse than "!commands mail commands":

"via mail commands, like !subscribe or !close"

24

Maybe explicitly mention DNS in this step?

25

"in your configuration"

35–36

This just subtracts hope. I'd skip it, especially since we already call out local MTA as being tricksy below.

57

"(e.g., the user taking the action)"

85

"create"

95

"applications"

src/docs/user/configuration/configuring_outbound_email.diviner
29–30

"This workplace has not an API change in 1 days."

This revision is now accepted and ready to land.Jan 16 2019, 9:45 PM
This revision was automatically updated to reflect the committed changes.

Oh, I totally missed that feedback, I'll followup.