Conversation
fixes #6958 Currently if you try to change the role of a Team member who's role is managed by a SSO group it silently fails. This adds an alert toast to explain why
|
reverting to draft as I've not actually read @knolleary's comments on the issue about doing it properly (which explains why he up'd it to 2 hours) |
|
@knolleary I was trying to knock this out quickly without looking too closely at your earlier suggested route.
I've now gone back to have another look and there is a small niggle with your approach. So to get SSO managed teams we would need to filter this list based on the SSO prefix/suffix settings to get just the team names (and remove the extra groups) Any thoughts? |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6994 +/- ##
=======================================
Coverage 76.29% 76.29%
=======================================
Files 403 403
Lines 20296 20296
Branches 4881 4881
=======================================
Hits 15485 15485
Misses 4811 4811
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I see 2 approaches here:
1 means we don't need to add anything else to the user object, but does mean always populating the SSOGroups array for a SSO user
|
fixes #6958
Description
Currently if you try to change the role of a Team member who's role is managed by a SSO group it silently fails.
This adds an alert toast to explain why
Related Issue(s)
#6958
Checklist
flowforge.yml?FlowFuse/helmto update ConfigMap TemplateFlowFuse/CloudProjectto update values for Staging/ProductionLabels
area:migrationlabel