Paths

Table of Contentst

Differential D20013

Make the "Test" adapter support both SMS and email
ClosedPublic
Actions

Authored by epriestley on Jan 21 2019, 11:17 PM.

Tags

None

Referenced Files

	F13087258: D20013.diff
	Thu, Apr 25, 12:55 AM

	F13080494: D20013.diff
	Wed, Apr 24, 7:44 AM

	Unknown Object (File)
	Sat, Apr 20, 4:29 PM

	Unknown Object (File)
	Thu, Apr 11, 8:41 AM

	Unknown Object (File)
	Sun, Apr 7, 10:55 PM

	Unknown Object (File)
	Sun, Mar 31, 12:56 PM

	Unknown Object (File)
	Mar 23 2024, 5:46 PM

	Unknown Object (File)
	Mar 23 2024, 5:46 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T920: Provide SMS Support

Commits

rPf69fbf5ea6d0: Make the "Test" adapter support both SMS and email

Summary

Depends on D20012. Ref T920. If you have a test adapter configured, it should swallow messages and prevent them from ever hitting a lower-priority adapter.

Make the test adapter support SMS so this actually happens.

Test Plan

Ran bin/mail send-test --type sms ... with a test adapter (first) and a Twilio adapter (second). Got SMS swallowed by test adapter instead of live SMS messages.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 21 2019, 11:17 PM

Harbormaster completed remote builds in B21633: Diff 47777.Jan 21 2019, 11:19 PM

epriestley requested review of this revision.Jan 21 2019, 11:19 PM

amckinley accepted this revision.Jan 22 2019, 12:08 AM

This revision is now accepted and ready to land.Jan 22 2019, 12:08 AM

epriestley added a child revision: D20016: Correct an issue where the default "Settings" screen could show the wrong settings.Jan 23 2019, 2:46 PM

Closed by commit rPf69fbf5ea6d0: Make the "Test" adapter support both SMS and email (authored by epriestley). · Explain WhyJan 23 2019, 10:07 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

adapter/

PhabricatorMailTestAdapter.php

38 lines

Diff 47812

src/applications/metamta/adapter/PhabricatorMailTestAdapter.php

Loading...