-
Notifications
You must be signed in to change notification settings - Fork 66
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hide the invite member button for users that don't have access to a team or insufficient privileges #4303
Hide the invite member button for users that don't have access to a team or insufficient privileges #4303
Conversation
…eam or insufficient privileges
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4303 +/- ##
=======================================
Coverage 78.19% 78.19%
=======================================
Files 292 292
Lines 13440 13440
Branches 3007 3007
=======================================
Hits 10510 10510
Misses 2930 2930
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
verified on pre-staging (logged in as owner, member and viewer & verified correct visibility)
merge when tests go green. |
…t in opening or showing an empty menu) - display the right user navigation button when no team present while hiding teams selector when no teams are available - hide invite members button on mobile devices to preserve nav width integrity - display the right notifications button on mobile - ui tweaks to the NoTeamUSer component that is being shown when a user does not have a team - hide the team selector 'create team' button if the platform settings do not allow users to create teams
…users-without-teams' into 4284_hide-invite-member-button-for-users-without-permissions # Conflicts: # frontend/src/components/PageHeader.vue
…or-users-without-permissions' into 4284_hide-invite-member-button-for-users-without-permissions
the ui failing test is the devices/bulk.spec.js. Will merge if ok with you, @Steve-Mcl! |
Yes please, then re-run these ui tests. |
Description
Hides the Invite member button for users that don't have sufficient privileges or insufficient privileges
Related Issue(s)
#4284
Checklist
flowforge.yml
?FlowFuse/helm
to update ConfigMap TemplateFlowFuse/CloudProject
to update values for Staging/ProductionLabels
area:migration
label