search

uselagoon / lagoon-ui

Apache License 2.0
6 stars 2 forks source link

feat: organization admin support #262

Closed shreddedbacon closed 4 hours ago

shreddedbacon commented 4 weeks ago

WIP: initial support for https://github.com/uselagoon/lagoon/pull/3733 !!! This will not work until the above PR is merged

tobybellwood commented 1 week ago

This will need a little work - I couldn't add admins via the UI - but the underlying GraphQL in the API does work. Also, "admin" should be an option when editing roles via the UI as well as owner.

shreddedbacon commented 1 week ago

This will need a little work - I couldn't add admins via the UI - but the underlying GraphQL in the API does work. Also, "admin" should be an option when editing roles via the UI as well as owner.

Yeah, I have no idea what I'm doing in the UI at this point, I just slapped this together as a starting point :D happy for someone to take it over, or ditch it entirely and start from scratch

DaveDarsa commented 4 days ago

@shreddedbacon I take it a user is either an owner or an admin since passing both booleans to the mutation causes the admin to be set to null and owner to true

with that in mind, the checkboxes behave mutually exclusively ( checking one unchecks the other )

shreddedbacon commented 3 days ago

@shreddedbacon I take it a user is either an owner or an admin since passing both booleans to the mutation causes the admin to be set to null and owner to true

with that in mind, the checkboxes behave mutually exclusively ( checking one unchecks the other )

Can you make it a dropdown box with 3 options ?

shreddedbacon commented 3 days ago

image Yeah, this is heaaapps better :partying_face: