Paths

Table of Contentst

Differential D20032

Expand outbound mailer documentation to mention SMS and include Twilio
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Dec 24, 5:22 PM

	Unknown Object (File)
	Tue, Dec 24, 4:13 PM

	Unknown Object (File)
	Sun, Dec 15, 1:19 AM

	Unknown Object (File)
	Sat, Dec 14, 3:03 PM

	Unknown Object (File)
	Sat, Dec 14, 7:19 AM

	Unknown Object (File)
	Sat, Dec 14, 5:25 AM

	Unknown Object (File)
	Fri, Dec 13, 7:10 PM

	Unknown Object (File)
	Fri, Dec 13, 10:28 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13222: 2018 Week 48-51 Bonus Content

Commits

rP50abc873630b: Expand outbound mailer documentation to mention SMS and include Twilio

Summary

Depends on D20031. Ref T13222.

Test Plan

Read "carefully".

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 25 2019, 3:12 PM

Harbormaster completed remote builds in B21694: Diff 47840.Jan 25 2019, 3:14 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.

Minor wordsmith.

Harbormaster completed remote builds in B21695: Diff 47841.Jan 25 2019, 3:17 PM

epriestley added a child revision: D20033: Update documentation for MFA, including administrator guidance.Jan 25 2019, 5:32 PM

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.

Reassure users that they can ignore the mailer warning if they don't plan to configure mailers.

Harbormaster completed remote builds in B21704: Diff 47850.Jan 26 2019, 7:34 PM

Closed by commit rP50abc873630b: Expand outbound mailer documentation to mention SMS and include Twilio (authored by epriestley). · Explain WhyJan 28 2019, 5:32 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

docs/

user/

configuration/

configuring_outbound_email.diviner

144 lines

Diff 47860

src/docs/user/configuration/configuring_outbound_email.diviner

Loading...