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

	F15517715: D20032.id47840.diff
	Sat, Apr 19, 7:59 AM

	F15509672: D20032.id47860.diff
	Wed, Apr 16, 3:08 PM

	F15506790: D20032.id.diff
	Tue, Apr 15, 2:13 PM

	F15502812: D20032.id47840.diff
	Mon, Apr 14, 4:37 AM

	F15479983: D20032.id47850.diff
	Tue, Apr 8, 10:41 AM

	F15477028: D20032.id47841.diff
	Mon, Apr 7, 11:33 AM

	F15468359: D20032.diff
	Thu, Apr 3, 11:47 PM

	F15456626: D20032.diff
	Sun, Mar 30, 10:51 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

Branch

mfa2

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21695
Build 29591: Run Core Tests
Build 29590: arc lint + arc unit

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
359	And maybe "Note: this will generate a Setup Warning which you are free to ignore"?
376	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
359	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

140 lines

Diff 47841

src/docs/user/configuration/configuring_outbound_email.diviner

Loading...