Page MenuHomePhabricator

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

Authored by matthewrbowker on Mar 3 2021, 2:22 AM.
Tags
None
Referenced Files
F12849663: D21587.diff
Fri, Mar 29, 4:45 AM
Unknown Object (File)
Mon, Mar 11, 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
Subscribers

Details

Summary
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

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
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.

src/applications/people/management/PhabricatorPeopleManagementApproveWorkflow.php
5

Mark this class as final.

16

"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