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
F13246568: D20725.diff
Thu, May 23, 11:33 AM
F13209500: D20725.id.diff
Fri, May 17, 12:35 AM
Unknown Object (File)
Tue, May 7, 5:28 AM
Unknown Object (File)
May 3 2024, 7:43 PM
Unknown Object (File)
May 3 2024, 2:54 AM
Unknown Object (File)
Apr 29 2024, 2:35 PM
Unknown Object (File)
Apr 24 2024, 10:40 PM
Unknown Object (File)
Apr 22 2024, 11:25 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.