-
-
Notifications
You must be signed in to change notification settings - Fork 870
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
Fix MFA auth flow #8720
Fix MFA auth flow #8720
Conversation
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
This should work as-is but waiting for feedback from @sur5r before marking ready |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8720 +/- ##
==========================================
- Coverage 85.24% 85.11% -0.14%
==========================================
Files 1175 1175
Lines 52424 52530 +106
Branches 2075 2075
==========================================
+ Hits 44691 44709 +18
- Misses 7226 7314 +88
Partials 507 507
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Got positive feedback in #8690 (comment) |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
* Re-add html account base Fixes inventree#8690 * fix base template
* Re-add html account base Fixes #8690 * fix base template Co-authored-by: Matthias Mair <[email protected]>
Re-adds (simplified) template to render MFA flows.
Closes #8690