Page MenuHomePhabricator

Align web UI "Disable" and "Approve/Disapprove" flows with new "Can Disable Users" permission
ClosedPublic

Authored by epriestley on Aug 27 2018, 2:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Feb 21, 7:59 AM
Unknown Object (File)
Tue, Feb 18, 3:10 PM
Unknown Object (File)
Sun, Feb 9, 6:22 AM
Unknown Object (File)
Sun, Feb 9, 6:22 AM
Unknown Object (File)
Sun, Feb 9, 6:22 AM
Unknown Object (File)
Fri, Feb 7, 4:42 PM
Unknown Object (File)
Fri, Feb 7, 3:38 PM
Unknown Object (File)
Tue, Feb 4, 3:09 AM
Subscribers
None

Details

Summary

Depends on D19606. Ref T13189. See PHI642.

  • Disabling/enabling users no longer requires admin. Now, you just need "Can Disable Users".
  • Update the UI to appropriately show the action in black or grey depending on what clicking the button will do.
  • For "Approve/Disapprove", fix a couple bugs, then let them go through without respect for "Can Disable Users". This is conceptually a different action, even though it ultimately sets the "Disabled" flag.
Test Plan
  • Disabled/enabled users from the web UI as various users, including a non-administrator with "Can Disable Users".
  • Hit permissions errors from the web UI as various users, including an administrator without "Can Disable Users".
  • Saw the "Disable/Enable" action activate properly based on whether clicking the button would actually work.
  • Disapproved a user without "Can Disable Users" permission, tried to re-disapprove a user.
  • Approved a user, tried to reapprove a user.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable