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
F19180726: D20010.diff
Dec 16 2025, 2:18 AM
F19069113: D20010.id.diff
Nov 30 2025, 1:05 PM
F19064453: D20010.diff
Nov 29 2025, 9:46 PM
F18844096: D20010.id47774.diff
Oct 29 2025, 3:05 AM
F18842186: D20010.id.diff
Oct 28 2025, 1:46 PM
F18840161: D20010.diff
Oct 27 2025, 9:56 PM
F18829576: D20010.diff
Oct 24 2025, 10:58 PM
F18826157: D20010.diff
Oct 24 2025, 2:47 AM
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
Branch
mfa7
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21635
Build 29500: Run Core Tests
Build 29499: arc lint + arc unit

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 47779

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