Skip to content

Conversation

@bbearce
Copy link
Collaborator

@bbearce bbearce commented Apr 11, 2023

Template

We want users to be able to change their passwords.

@ mention of reviewers

@Didayolo
@dtuantran
@anne-catherine

A brief description of the purpose of the changes contained in this PR.

We want users to be able to change their passwords. This includes a forgot password functionality and change password functionality. Patterned after codalab this allows either use case to be funneled into a change password functionality with email token sent to verify user account.

Issues this PR resolves

#759...partially at least.
#706

A checklist for hand testing

Any relevant files for testing

Checklist

  • [ x] Code review by me
  • [ x] Hand tested by me
  • [ x] I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

@Didayolo
Copy link
Member

@bbearce

Flake8 formatting issues:

^@^@src/apps/profiles/urls_accounts.py:15:1: W293 blank line contains whitespace
src/apps/profiles/views.py:163:1: W293 blank line contains whitespace
src/apps/profiles/views.py:166:1: E302 expected 2 blank lines, found 1
src/apps/profiles/views.py:175:110: W291 trailing whitespace
src/apps/profiles/views.py:179:41: E261 at least two spaces before inline comment

@Didayolo Didayolo requested a review from ihsaan-ullah April 19, 2023 13:31
@Didayolo Didayolo merged commit 7de11ea into develop Apr 19, 2023
@Didayolo Didayolo deleted the forgot_change_password branch April 19, 2023 13:43
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.

3 participants