-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Force users to accept the privacy policy #3803
Conversation
This pull request introduces 1 alert when merging b0630d9 into e316995 - view on LGTM.com new alerts:
|
…na into feature/terms-of-service
The way this is implemented right now this check happens after sign in (Or after account creation) While writing the new privacy policy, I have come to the realization that it should probably be part of the sign up process. This way no account would be created if the privacy policy is not accepted.. I'll update this |
closed in favor of #3859 |
This pull request forces users to accept the privacy policy before they can use dodona.
On most pages users are redirected to this prompt, unless they have already accepted it:
Open questions:
Part of #2597