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

Feature to reset account password via email verification #245

Merged
merged 25 commits into from
Oct 5, 2023

Conversation

jackson-hu1279
Copy link
Collaborator

Users can submit a reset password request via the forget password link on the login page now. A verification email with a password reset link will be sent to the given email address linked to the account allowing password reset.

The feature is added under the myCSSAhub module and details are as follows:

URLs added:

  • password_reset/ - reset request form page
  • password_reset_sent/ - email sent confirmation page
  • password_reset/<uidb64>/<token>/ - set new password page
  • password_reset_complete/ - password reset confirmation page

Views added:

  • PasswordResetView - main logic to validate and send email
  • PasswordResetSentView - page redirct
  • PasswordResetConfirmView (django auth views) - set new password
  • PasswordResetCompleteView - page redirect

HTML templates added/modified:

  • myCSSAhub/login.html
  • myCSSAhub/password_reset.html
  • myCSSAhub/password_reset_sent.html
  • myCSSAhub/password_reset_confirm.html
  • myCSSAhub/password_reset_complete.html

@jackson-hu1279 jackson-hu1279 added New-Feature New feature or request High-Priority Must done within the assigned sprint labels Sep 16, 2023
Copy link
Collaborator

@doncd-p doncd-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Environment variables for automail changed to [email protected] with corresponding password.

Copy link
Collaborator

@doncd-p doncd-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test passed under dev server.
image

@doncd-p doncd-p merged commit 424f4ec into master Oct 5, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
High-Priority Must done within the assigned sprint New-Feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants