Page MenuHomePhabricator

Remove "bin/accountadmin" and "scripts/user/add_user.php"
ClosedPublic

Authored by epriestley on Aug 21 2019, 12:45 AM.
Tags
None
Referenced Files
F15514415: D20725.id.diff
Fri, Apr 18, 2:30 AM
F15511982: D20725.diff
Thu, Apr 17, 7:09 AM
F15506470: D20725.diff
Tue, Apr 15, 11:31 AM
F15484096: D20725.id49412.diff
Wed, Apr 9, 4:10 PM
F15480161: D20725.id49410.diff
Tue, Apr 8, 12:20 PM
F15457585: D20725.diff
Sun, Mar 30, 5:18 PM
F15388051: D20725.id49410.diff
Mar 15 2025, 3:15 AM
F15386125: D20725.diff
Mar 15 2025, 12:05 AM
Subscribers
None

Details

Summary

Fixes T13382. Depends on D20724. These ancient scripts are no longer necessary since we've had a smooth web-based onboarding process for a long time.

I retained bin/user empower and bin/user enable for recovering from situations where you accidentally delete or disable all administrators. This is normally difficult, but some users are industrious.

Test Plan

Grepped for accountadmin and add_user.php, found no more hits.

Diff Detail

Repository
rP Phabricator
Branch
user2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 23271
Build 31974: Run Core Tests
Build 31973: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Aug 21 2019, 12:58 AM
This revision was automatically updated to reflect the committed changes.