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)
Sun, Apr 14, 1:12 PM
Unknown Object (File)
Sat, Apr 6, 12:18 AM
Unknown Object (File)
Mar 4 2024, 6:27 PM
Unknown Object (File)
Feb 3 2024, 7:17 PM
Unknown Object (File)
Jan 2 2024, 8:17 AM
Unknown Object (File)
Nov 30 2023, 12:35 AM
Unknown Object (File)
Nov 15 2023, 8:08 AM
Unknown Object (File)
Oct 3 2023, 3:56 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