Page MenuHomePhabricator

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

Authored by epriestley on Aug 21 2019, 12:45 AM.

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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Aug 21 2019, 12:45 AM
epriestley requested review of this revision.Aug 21 2019, 12:47 AM
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.