Skip to content
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

Add option to disable frontdoor login #723

Merged
merged 4 commits into from
Feb 10, 2024
Merged

Add option to disable frontdoor login #723

merged 4 commits into from
Feb 10, 2024

Conversation

paustint
Copy link
Contributor

@paustint paustint commented Feb 8, 2024

Some users have issues with multi-factor authentication when using frontdoor login, so we are offering an option to disable it globally for a user.

Added new user preferences DB table to handle all interactions

Frontdoor login honors DB setting for most cases, for places where it was difficult to access the user profile this is disabled for all users to provide as consistent experience as possible

TODO: Additional testing across all surfaces

resolves #716

image

Some users have issues with multi-factor authentication when using frontdoor login, so we are offering an option to disable it globally for a user.

Added new user preferences DB table to handle all interactions

resolves #716
Copy link

nx-cloud bot commented Feb 8, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 7b81c77. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

Some datatable actions were not honoring the profile setting

Fixed issue where skipping frontdoor caused an extra `/` in the URL
@paustint paustint merged commit 8305758 into main Feb 10, 2024
5 checks passed
@paustint paustint deleted the feat/716 branch November 23, 2024 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Choose a Verification Method
1 participant