Skip to content
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

Update send_verification_email and verify_email_and_activate #85

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JjayFabor
Copy link

I don't know if this function is already fix and I did not understand to use this framework well so, in my case, I have modified the send_verification_email function in the django-verify-email package to allow sending verification emails directly using the user object when no form is provided. This enhancement is useful for projects using Django as the backend with a different framework, like React.

I have also modified the verify_user_and_activate function that causes me NoReverseMatch error when I don't have a login_page. The function has been modified to handle this issue that I encounter.

Why This Contribution is Valuable:

This feature increases the flexibility of django-verify-email by allowing developers to use it in headless Django setups. It simplifies integration for projects using different frontend frameworks.

Thank you for considering my contribution. Please let me know if there are any changes or additional information needed.

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.

1 participant