-
Notifications
You must be signed in to change notification settings - Fork 446
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
Enable MFA checkbox is always greyed out for team admins #24623
Comments
It looks like the above is intentional based on this comment: |
@noahtalerman, to over communicate, we do want team admins to be able to enable MFA like this right? If so, do we want to expose the org-level smtp settings to team admins for the purpose of this UI? Another idea, we could just have the UI allow it if it doesn't know the smtp settings and the server error if they aren't set. cc @iansltx @lucasmrod (left the comment above) |
another idea – just expose |
Chatted with @noahtalerman and for this iteration we'll stay consistent with the existing UI...which doesn't expose any advanced user management (SSO, MFA, invites) for team admins. This represents a gap between what the API can do and what the UI can do, and I'll get a bug up covering that delta (which will require BE work to expose both SMTP and SSO enableement flags to team admins so we en/disable UI controls properly). So the fix for this particular ticket is to drop the checkbox entirely for team admins, as a FE-only change. |
Also fixes some capitalization/pluralization issues for user management for team admins For #24623.
QA plan /cc @jmwatts:
|
Also fixes some capitalization/pluralization issues for user management for team admins For #24623. # Checklist for submitter - [x] Manual QA for all new/changed functionality
Also fixes some capitalization/pluralization issues for user management for team admins For #24623. # Checklist for submitter - [x] Manual QA for all new/changed functionality
@iansltx Just want to confirm that Team admins shouldn't be able to see if another user has 2FA enabled or not. ALSO - |
Re: viewing user info, correct; neither SSO status nor MFA status are expected to be shown for users when viewed/editing with a team admin. Re: creating with MFA using the API, that's expected. The API is correct here, and #24660 will fix the gap between UI and API (and will get worked on sooner than #24465). |
QA Note: Verified team admins no longer see MFA checkbox for create/edit user per comments above. |
Greyed out box unsealed, |
Fleet version: v4.61.0
Web browser and operating system: Chrome 131.0.6778.109 running on macOS
💥 Actual behavior
🧑💻 Steps to reproduce
Ensure SMTP is enabled
🕯️ Expected behavior
The team admin should be able to enable 2FA for other users on their team
The text was updated successfully, but these errors were encountered: