-
Notifications
You must be signed in to change notification settings - Fork 3
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
Feature/189 install maykin 2fa #194
Conversation
1e3232b
to
699d5bc
Compare
Requires maykinmedia/open-api-framework#27 to be merged and released |
Requires maykinmedia/open-api-framework#31 to be merged and released. |
Waiting for Other PR #190 to merge Merge Complete |
64f0563
to
f2f4b03
Compare
f2f4b03
to
056fbab
Compare
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.
Works as expected, could you also add DISABLE_2FA
to the docker-compose envvars (with default True
)?
I do notice that some templates show the admin version though, these should not be visible until the user is fully authenticated
after adding 2FA:
backuptokens template:
I think this can be fixed by overriding the base template footer: https://github.com/open-zaak/open-zaak/blob/main/src/openzaak/templates/maykin_2fa/base.html
ff87eed
to
3c73226
Compare
Fixes #189