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
F17811989: D19607.id.diff
Fri, Jul 25, 7:01 PM
F17764278: D19607.diff
Wed, Jul 23, 12:30 AM
Unknown Object (File)
Jun 17 2025, 10:42 AM
Unknown Object (File)
Jun 16 2025, 5:46 PM
Unknown Object (File)
Jun 5 2025, 5:10 PM
Unknown Object (File)
Jun 2 2025, 8:28 PM
Unknown Object (File)
Jun 2 2025, 8:28 PM
Unknown Object (File)
Jun 2 2025, 8:22 PM
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