Skip to content

feat: Redirect users to profile if not agreed to TOS #125

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 6 commits into
base: main
Choose a base branch
from

Conversation

Mr-Sunglasses
Copy link
Contributor

fix: #121

Copy link

netlify bot commented May 15, 2025

👷 Deploy request for pyladiescon-portal-docs accepted.

Name Link
🔨 Latest commit a35f6db
🔍 Latest deploy log https://app.netlify.com/projects/pyladiescon-portal-docs/deploys/682b8d90aeab6b00085107d2

@Mr-Sunglasses
Copy link
Contributor Author

Requesting @Mariatta for a review.

@Mariatta
Copy link
Member

Thanks for the PR. I kinda wish to use the route of using the UserPassesTestMixin instead of a middleware.
https://docs.djangoproject.com/en/5.2/topics/auth/default/#django.contrib.auth.mixins.UserPassesTestMixin

@Mr-Sunglasses
Copy link
Contributor Author

Thanks for the PR. I kinda wish to use the route of using the UserPassesTestMixin instead of a middleware.

https://docs.djangoproject.com/en/5.2/topics/auth/default/#django.contrib.auth.mixins.UserPassesTestMixin

Should I rewrite it with UserPassesTestMixin or the current approach with middleware work?

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.

Redirect users to complete their profile if they have not agreed to TOS
2 participants