Page MenuHomePhabricator

Implement most of the administrative UI for approval queues
ClosedPublic

Authored by epriestley on Nov 13 2013, 2:27 AM.
Tags
None
Referenced Files
F13422115: D7573.diff
Mon, Jul 8, 6:01 PM
F13404119: D7573.diff
Fri, Jul 5, 12:56 PM
F13396235: D7573.id17105.diff
Thu, Jul 4, 12:51 AM
F13367597: D7573.id17094.diff
Thu, Jun 27, 2:09 AM
F13366055: D7573.id17105.diff
Wed, Jun 26, 6:08 PM
F13350252: D7573.diff
Sun, Jun 23, 6:26 AM
F13318596: D7573.diff
Thu, Jun 13, 11:16 AM
F13307173: D7573.diff
Sun, Jun 9, 2:31 PM
Subscribers

Details

Summary

Nothing fancy here, just:

  • UI to show users needing approval.
  • "Approve" and "Disable" actions.
  • Send "Approved" email on approve.
  • "Approve" edit + log operations.
  • "Wait for Approval" state for users who need approval.

There's still no natural way for users to end up not-approved -- you have to write directly to the database.

Test Plan

See screenshots.

Diff Detail

Branch
appqueue
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

{F81257}

{F81258}

{F81259}

{F81260}

{F81261}

{F81262}

btrahan added inline comments.
src/applications/auth/controller/PhabricatorAuthNeedsApprovalController.php
26

haha

src/applications/people/editor/PhabricatorUserEditor.php
307

no user editor?

src/applications/auth/controller/PhabricatorAuthNeedsApprovalController.php
26

This might get us an "IT'S NOT SERIOUS ENOUGH" email, but the dialog currently looks junky with no buttons and implementing this button amused me greatly.

src/applications/people/editor/PhabricatorUserEditor.php
307

This is PhabricatorUserEditor, but it predates ApplicationTransactions so it's kind of funky and has a lot of code duplication compared to ApplicationTransactions (no shared code for database transaction management, e.g). We'll probably switch it over eventually, but I think it's lowish priority since it doesn't cause any concrete problems or really block anything (e.g., we don't need to publish any of this to news feed).

Well, maybe it would be nice to publish a "user signed up" story.

And maybe it would be nice to get a Herald email on a new user signup.

So maybe this does block stuff.

But no one has asked for these things, at least.