Page MenuHomePhabricator

Show primary email in approval queue interface
ClosedPublic

Authored by epriestley on Mar 21 2014, 9:29 PM.
Tags
None
Referenced Files
F14056890: D8589.diff
Sat, Nov 16, 11:10 PM
F14035873: D8589.id20369.diff
Sun, Nov 10, 7:51 AM
F14004568: D8589.id.diff
Sat, Oct 26, 11:07 PM
F13992808: D8589.diff
Tue, Oct 22, 6:56 PM
F13992795: D8589.id.diff
Tue, Oct 22, 6:49 PM
F13992793: D8589.id20369.diff
Tue, Oct 22, 6:48 PM
F13992789: D8589.id20417.diff
Tue, Oct 22, 6:47 PM
F13956720: D8589.id.diff
Oct 14 2024, 6:51 AM

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rP62e3463ab791: Show primary email in approval queue interface
Summary

Request from @csilvers. When approving users, the primary email address is useful for administrators.

(This queue is only accessible by administrators, so this doesn't expose email information in general.)

Test Plan

Screen_Shot_2014-03-21_at_2.27.41_PM.png (627×1 px, 114 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Show primary email in approval queue interface.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a subscriber: csilvers.

Maybe only expose this to admins? Or is it implicit that you cannot see this view unless you have the ability to make approvals?

Yeah --

(This queue is only accessible by administrators, so this doesn't expose email information in general.)

Users are only visible on this queue if they're pending approval, and only administrators can see it.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 25 2014, 7:39 PM
epriestley updated this revision to Diff 20417.

Closed by commit rP62e3463ab791 (authored by @epriestley).