Page MenuHomePhabricator
Differential D20010

Allow contact numbers to be enabled and disabled
ClosedPublic
Actions

Authored by epriestley on Jan 21 2019, 8:52 PM.
Tags
None
Referenced Files
F15517345: D20010.id47779.diff
Sat, Apr 19, 2:16 AM
F15515593: D20010.id47779.diff
Fri, Apr 18, 11:41 AM
F15515469: D20010.id.diff
Fri, Apr 18, 10:39 AM
F15515359: D20010.diff
Fri, Apr 18, 9:51 AM
F15507168: D20010.diff
Tue, Apr 15, 4:58 PM
F15505759: D20010.id47809.diff
Tue, Apr 15, 5:47 AM
F15505000: D20010.id47779.diff
Mon, Apr 14, 11:17 PM
F15504337: D20010.diff
Mon, Apr 14, 5:31 PM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T920: Provide SMS Support
Commits
rP12203762b7f0: Allow contact numbers to be enabled and disabled
Summary

Depends on D20008. Ref T920. Continue fleshing out contact number behaviors.

Test Plan
  • Enabled and disabled a contact number.
  • Saw list, detail views reflect change.
  • Added number X, disabled it, added it again (allowed), enabled the disabled one ("already in use" exception).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 21 2019, 8:52 PM
Harbormaster completed remote builds in B21630: Diff 47774.Jan 21 2019, 8:54 PM
epriestley requested review of this revision.Jan 21 2019, 8:54 PM
epriestley added a child revision: D20011: Support designating a contact number as "primary".Jan 21 2019, 10:32 PM
amckinley accepted this revision.Jan 21 2019, 10:40 PM
amckinley added inline comments.
src/applications/auth/xaction/PhabricatorAuthContactNumberStatusTransaction.php
18

Unused variable.

This revision is now accepted and ready to land.Jan 21 2019, 10:40 PM
epriestley updated this revision to Diff 47779.Jan 22 2019, 12:01 AM
  • Prune unused variable.
Harbormaster completed remote builds in B21635: Diff 47779.Jan 22 2019, 12:02 AM
Closed by commit rP12203762b7f0: Allow contact numbers to be enabled and disabled (authored by epriestley). · Explain WhyJan 23 2019, 10:00 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 47809

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/auth/application/PhabricatorAuthApplication.php

Loading...
View Options

src/applications/auth/controller/contact/PhabricatorAuthContactNumberDisableController.php

Loading...
View Options

src/applications/auth/controller/contact/PhabricatorAuthContactNumberViewController.php

Loading...
View Options

src/applications/auth/storage/PhabricatorAuthContactNumber.php

Loading...
View Options

src/applications/auth/xaction/PhabricatorAuthContactNumberStatusTransaction.php

Loading...
Phabricator · Built by Phacility