Page MenuHomePhabricator

Update inbound and outbound email documentation
ClosedPublic

Authored by epriestley on Jan 14 2019, 11:21 PM.
Tags
None
Referenced Files
F18822121: D19973.id47673.diff
Thu, Oct 23, 4:22 AM
F18817734: D19973.id.diff
Tue, Oct 21, 5:55 PM
F18809124: D19973.id.diff
Sun, Oct 19, 11:22 AM
F18806887: D19973.diff
Sat, Oct 18, 9:55 PM
F18789384: D19973.diff
Wed, Oct 15, 9:26 AM
F18788120: D19973.id47672.diff
Wed, Oct 15, 2:54 AM
F18765044: D19973.diff
Tue, Oct 7, 9:14 AM
F18748899: D19973.diff
Sat, Oct 4, 1:23 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
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.