Skip to content
This repository has been archived by the owner on Mar 8, 2020. It is now read-only.

WIP password change: routes and view. Not working yet. #13

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JessicaKCao
Copy link

@JessicaKCao JessicaKCao commented Feb 28, 2019

Resolves #4

Description

The feature is now working and has a feature test. Let me know if there are concerns or questions.

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update - not sure. is there user-focused documentation on login/account management that would need to be updated?

How Has This Been Tested?

The feature test is readable and easy to understand what is being tested. The test first creates an account, goes to update the partner's password, and check if the new password is valid while making sure that the old password no longers works.

To test manually:

  1. Login
  2. Click "Edit Partner"
  3. Scroll to the bottom and click "Change your password" (bottom left)
  4. Enter the requested information and click "Update"
  5. You should be redirected to the "My Organization" page and see a green box at the top that says "Your account has been successfully updated"
  6. Logout, and try logging in with the new password

Screenshots

@JessicaKCao JessicaKCao self-assigned this Feb 28, 2019
@JessicaKCao JessicaKCao requested a review from mdworken February 28, 2019 19:18
@JessicaKCao
Copy link
Author

@mdworken can you please review again?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[2:3:3] Allow a partner to update/change their password
2 participants