164798252 users should be able to reset their passwords #17

anyatibrian · 2019-05-16T16:57:35Z

What does this PR do?

Enable users to be able to reset their password in case they have forgotten

Description of tasks to be completed.

Add tests for a password reset functionality
send a link to the user's email with their password reset token
Add pages send an email and resetting password respectively

How should this be manually tested?

Visit the deployed application page on heroku
enter your email that you had registered with in the app,then you will recieve an email with a password reset link, follow this link that will redirect to the password reset page from where you enter and confirm your new password.

Any background context you want to provide?

N/A

What are the relevant pivotal tracker stories?
#164798252

Screen shots

send password resetlink page

password reset page

src/styles/passwordreset.scss

src/reducers/passwordResetReducer.js

src/reducers/passwordChangeReducer.js

src/containers/passwordResetEmail.js

src/containers/passwordReset.js

src/components/passwordResetEmail.js

src/components/passwordReset.js

src/actions/passwordResetActions.js

src/styles/register.scss

e-ian · 2019-05-16T17:31:33Z

@anyatibrian I see you have three commits, please squash them into one commit

src/styles/register.scss

src/utils/validateforms.js

src/utils/regex.js

src/urls.js

src/tests/SocialAuth/SocialAuthReducer.spec.js

src/tests/SocialAuth/SocialAuthContainer.spec.js

src/styles/register.scss

anyatibrian · 2019-05-18T08:51:42Z

Nice work @anyatibrian , the functionality works as expected. Please ensure to:
* resolve merge conflicts

* fully test the `passwordReset.js` and `passwordResetEmail.js` files.

am done with the project

patrickf949

Tested
Feedback implemented
LGTM

MuhanguziDavid

Nice work, however, as we talked about earlier, please add a statement to dismiss previous toasts in your action so as to avoid this

src/tests/components/passwordReset.test.js

src/tests/components/passwordResetEmail.test.js

src/tests/containers/passwordResetContainer.test.js

src/styles/sidedrawer.scss

src/styles/navbar.scss

anyatibrian · 2019-05-18T18:03:29Z

Nice work @anyatibrian , the functionality works as expected. Please ensure to:
* resolve merge conflicts

* fully test the `passwordReset.js` and `passwordResetEmail.js` files.

am done with the project

Nice work, however, as we talked about earlier, please add a statement to dismiss previous toasts in your action so as to avoid this

done

src/styles/navbar.scss

src/styles/app.scss

src/components/sideDrawer.js

src/components/navbar.js

- add test for the password reset feature - add views files for the feature - add validations for the form fields - add loaders for the feature [Finishes 164798252]

MuhanguziDavid

Awesome stuff @anyatibrian , LGTM

patrickf949 had a problem deploying to ah-frontend-prime-pr-17 May 16, 2019 16:57 Failure

houndci-bot reviewed May 16, 2019

View reviewed changes

anyatibrian force-pushed the ft-password-reset-164798252 branch from 8a6ca62 to c48f0fe Compare May 16, 2019 17:01

patrickf949 temporarily deployed to ah-frontend-prime-pr-17 May 16, 2019 17:01 Inactive

houndci-bot reviewed May 16, 2019

View reviewed changes

anyatibrian changed the title ~~Ft password reset 164798252~~ 164798252 users should be able to reset their passwords May 16, 2019

anyatibrian assigned anyatibrian, Genza999, e-ian, MuhanguziDavid and patrickf949 May 16, 2019

anyatibrian force-pushed the ft-password-reset-164798252 branch from c48f0fe to 42323a1 Compare May 16, 2019 18:40

patrickf949 temporarily deployed to ah-frontend-prime-pr-17 May 16, 2019 18:40 Inactive

houndci-bot reviewed May 16, 2019

View reviewed changes

anyatibrian force-pushed the ft-password-reset-164798252 branch from 42323a1 to 2f108dc Compare May 16, 2019 19:22

patrickf949 temporarily deployed to ah-frontend-prime-pr-17 May 16, 2019 19:22 Inactive

houndci-bot reviewed May 16, 2019

View reviewed changes

anyatibrian added the finished label May 17, 2019

anyatibrian requested review from e-ian, patrickf949, Genza999 and MuhanguziDavid May 17, 2019 06:33

anyatibrian unassigned anyatibrian, Genza999, e-ian, MuhanguziDavid and patrickf949 May 17, 2019

anyatibrian force-pushed the ft-password-reset-164798252 branch from e0235fb to 9e53601 Compare May 17, 2019 14:46

patrickf949 temporarily deployed to ah-frontend-prime-pr-17 May 17, 2019 14:46 Inactive

patrickf949 approved these changes May 18, 2019

View reviewed changes

MuhanguziDavid suggested changes May 18, 2019

View reviewed changes