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
F14376688: D7573.diff
Sat, Dec 21, 12:09 AM
Unknown Object (File)
Sat, Dec 14, 4:56 AM
Unknown Object (File)
Wed, Dec 11, 3:49 PM
Unknown Object (File)
Sat, Nov 30, 9:00 AM
Unknown Object (File)
Fri, Nov 29, 5:52 PM
Unknown Object (File)
Fri, Nov 29, 5:52 PM
Unknown Object (File)
Fri, Nov 29, 5:44 PM
Unknown Object (File)
Sat, Nov 23, 6:00 AM
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

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

{F81257}

{F81258}

{F81259}

{F81260}

{F81261}

{F81262}

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

haha

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

no user editor?

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

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.