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
Unknown Object (File)
Mon, Dec 30, 6:50 PM
Unknown Object (File)
Mon, Dec 30, 3:24 PM
Unknown Object (File)
Mon, Dec 23, 8:14 AM
Unknown Object (File)
Fri, Dec 20, 6:40 PM
Unknown Object (File)
Wed, Dec 18, 8:09 AM
Unknown Object (File)
Thu, Dec 12, 10:08 PM
Unknown Object (File)
Sat, Dec 7, 6:13 PM
Unknown Object (File)
Dec 3 2024, 12:49 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.