Page MenuHomePhabricator

Update inbound and outbound email documentation
ClosedPublic

Authored by epriestley on Jan 14 2019, 11:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 2:58 AM
Unknown Object (File)
Fri, Apr 5, 3:21 PM
Unknown Object (File)
Mar 23 2024, 3:27 AM
Unknown Object (File)
Jan 25 2024, 1:43 AM
Unknown Object (File)
Dec 23 2023, 10:12 PM
Unknown Object (File)
Dec 22 2023, 1:12 AM
Unknown Object (File)
Dec 14 2023, 1:59 AM
Unknown Object (File)
Dec 13 2023, 5:56 PM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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–31

"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.