Page MenuHomePhabricator

Add a "test message" action for contact numbers
ClosedPublic

Authored by epriestley on Jan 23 2019, 8:18 PM.

Details

Summary

Depends on D20024. See D20022. Put something in place temporarily until we build out validation at some point.

Test Plan

Sent myself a test message.

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 updated this revision to Diff 47799.Jan 23 2019, 8:18 PM
epriestley created this revision.
  • Wordsmithing.
Harbormaster completed remote builds in B21652: Diff 47799.
epriestley requested review of this revision.Jan 23 2019, 8:21 PM
epriestley updated this revision to Diff 47800.Jan 23 2019, 8:21 PM
  • Don't force delivery (so the mail page will show delivery issues).
  • Don't mark the content as sensitive (it's not particularly secret).
amckinley accepted this revision.Jan 23 2019, 8:26 PM
amckinley added inline comments.
src/applications/auth/action/PhabricatorAuthTestSMSAction.php
20

"textings"... or maybe this was supposed to be "testing texts"?

This revision is now accepted and ready to land.Jan 23 2019, 8:26 PM
epriestley updated this revision to Diff 47801.Jan 23 2019, 8:29 PM
  • Texting's
This revision was automatically updated to reflect the committed changes.