Page MenuHomePhabricator

Expand outbound mailer documentation to mention SMS and include Twilio
ClosedPublic

Authored by epriestley on Jan 25 2019, 3:12 PM.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Jan 25 2019, 3:12 PM
epriestley requested review of this revision.Jan 25 2019, 3:14 PM
epriestley added inline comments.Jan 25 2019, 3:16 PM
src/docs/user/configuration/configuring_outbound_email.diviner
211

(I tested that all of these http -> https changes actually work.)

278

I broke this out since SES doesn't actually require region, today.

epriestley updated this revision to Diff 47841.Jan 25 2019, 3:16 PM
  • Minor wordsmith.
amckinley accepted this revision.Jan 26 2019, 7:00 PM
amckinley added inline comments.
src/docs/user/configuration/configuring_outbound_email.diviner
12

Only for translation reasons, maybe:

"(like SMS messages to a mobile phone)."

280–281

You could link to here if you wanted: https://docs.aws.amazon.com/general/latest/gr/rande.html

353–361

And maybe "Note: this will generate a Setup Warning which you are free to ignore"?

378

And maybe note that there is an SMS test button on the Settings page?

This revision is now accepted and ready to land.Jan 26 2019, 7:00 PM
epriestley added inline comments.Jan 26 2019, 7:31 PM
src/docs/user/configuration/configuring_outbound_email.diviner
353–361

Oh, good call.

epriestley updated this revision to Diff 47850.Jan 26 2019, 7:32 PM
  • Reassure users that they can ignore the mailer warning if they don't plan to configure mailers.
This revision was automatically updated to reflect the committed changes.