#159054010 Password Reset Feature

[asamisellah]

What does this PR do?

Implement reset password with email verification

Description of Task to be completed?

• Sends verification email to the user for password reset
• Verify the user and updates the new user password

Endpoints

METHOD	ENDPOINT	FUNCTIONALITY
POST	/api/users/forgot_password	Sends reset password verification email
PUT	/api/users/resetpassword	Validates email and resets user password

How should this be manually tested?

• Clone this repo
• Create an account with sendgrid and acquire an apikey which you will add to the .env as follows
  EMAIL_HOST_PASSWORD=YOUR_SENDGRID_API_KEY

cd ah-jarvis

git pull origin ft-reset-password-159054010
  
create and activate a virtual env 

pip install -r requirements.txt

python manage.py migrate

coverage run manage.py test

python manage.py run server

• Launch Postman App.
• Register an account with your valid email as follows:

URL: http://127.0.0.1:8000/api/users/

{
  "user": {
	"username":"jake",
	"email":"validemail@example.com",
	"password":"Pass1234"
  }
}

• Post your valid email to get a Password Reset Validation email

URL: http://127.0.0.1:8000/api/users/forgot_password

{
  "email": "validemail@example.com"
}

• Open your email and find an email from webmaster@localhost
• Copy the last part of the URL(the token)
• Include the token as an input in the reset password endpoint as follows:

URL: http://127.0.0.1:8000/api/users/resetpassword

{
  "reset_data":{
  	"token": "token_sent_to_email",
  	"email": "validemail@example.com",
  	"new_password": "new_password"
  }
}

Any background context you want to provide?

None

What are the relevant pivotal tracker stories?

#159054010 Users should be able to reset password via email

[dwarukira]

Looks good. Need to resolve merge conflicts.

[coveralls]

Coverage decreased (-0.3%) to 96.569% when pulling f4211ef on ft-reset-password-159054010 into 7b27382 on develop.

[denniswanjiru]

Tried out the PR on my local machine and worked as expected. You need to resolve the conflicts, though.

[asamisellah]

@denniswanjiru Thanks for the feedback. I have resolved the merge conflicts