Page MenuHomePhabricator

Provide new tool "bin/user approve" to approve an account.

Authored by matthewrbowker on Mar 3 2021, 2:22 AM.
Referenced Files
Unknown Object (File)
Fri, Mar 29, 4:45 AM
Unknown Object (File)
Mar 11 2024, 1:12 PM
Unknown Object (File)
Feb 4 2024, 12:46 AM
Unknown Object (File)
Jan 27 2024, 8:31 PM
Unknown Object (File)
Jan 16 2024, 3:40 AM
Unknown Object (File)
Jan 9 2024, 7:58 AM
Unknown Object (File)
Dec 28 2023, 4:50 PM
Unknown Object (File)
Dec 26 2023, 7:22 PM


Test Plan

This method uses the existing transaction. As such, most of the testing focused on the integration between the workflow and transaction. The only change made to the transaction was to allow an omnipotent user to make the change in addition to an admin.

Other than that, I removed the "approved" flag from the user, then ran the command-line utilty until the user was successfully approved.

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

Couple minor inlines, but looks good to me. I added you to Blessed Committers so you should be able to land this yourself -- see the description of that project for guidance, or let me know if you run into issues.


Mark this class as final.


"Approve a user." (with a period) is a little more consistent with other commands.

This revision is now accepted and ready to land.Mar 3 2021, 3:25 AM
  • Address code review comments