theopensystemslab / planx-new

Plan✕ is a platform for creating and publishing digital planning services
https://editor.planx.uk
Mozilla Public License 2.0
13 stars 2 forks source link

fix: hide "Add a new editor" button from non-admin users #3713

Open jessicamcinchak opened 10 hours ago

jessicamcinchak commented 10 hours ago

It's the second planning meeting in a row where Ian G has asked about / been a bit confused by adding and editing users and I think it's because he can see the add button as a teamEditor, but doesn't have permission to actually use it.

This change hides the "Add a new editor" from non-platformAdmins similar to the "Edit" button. Database-level permissions already look correct (eg only platformAdmin role has "insert" permission on team_members table).

We've thrown this previous error: https://opensystemslab.slack.com/archives/C01E3AC0C03/p1725542317058179 Screenshot from 2024-09-19 15-38-11

Good reminder to ourselves to trigger our own platformAdmin status "off" sometimes when testing/code reviewing pizzas!

github-actions[bot] commented 10 hours ago

Pizza

Deployed b7ae9d815db859b4491f6d9d4ce4e9c6a6f78f81 to https://3713.planx.pizza.

Useful links: