Skip to content

feat(frontend): make mfa login faster #9595

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

matmair
Copy link
Contributor

@matmair matmair commented Apr 27, 2025

This PR visually removes the MFA step for users of most password managers.

This is done by adding a hidden field for the TOTP code to the authentication window and adapting the login flow to directly trigger the MFA flow.
The logical steps are the same. Nothing changes for users that manually input TOTP.

@matmair matmair added the User Interface Related to the frontend / User Interface label Apr 27, 2025
@matmair matmair self-assigned this Apr 27, 2025
@matmair matmair added this to the 1.0.0 milestone Apr 27, 2025
Copy link

netlify bot commented Apr 27, 2025

Deploy Preview for inventree-web-pui-preview ready!

Name Link
🔨 Latest commit 028ee29
🔍 Latest deploy log https://app.netlify.com/sites/inventree-web-pui-preview/deploys/680ebbad2e429800082c609a
😎 Deploy Preview https://deploy-preview-9595--inventree-web-pui-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 96 (no change from production)
Accessibility: 81 (🔴 down 8 from production)
Best Practices: 100 (no change from production)
SEO: 78 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
User Interface Related to the frontend / User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant