hngprojects / hng_boilerplate_python_fastapi_web

Apache License 2.0
154 stars 137 forks source link

fix: Removed email change from user and profile update, added separate route for recovery email update #945

Closed johnson-oragui closed 3 weeks ago

johnson-oragui commented 3 weeks ago

This Pull request aims to remove the change of email from /users and /profile endpoint. The change of user email is now moved to PUT /profile, and a user email can only be changed from email confirmation for security reason and to ensure data integrity. ​

Description

This Pull request aims to remove the change of email from /users and /profile endpoint. The change of user email is now moved to PUT /profile, and a user email can only be changed from email confirmation for security reason and to ensure data integrity.

Related Issue (Link to issue ticket)

[FIX] Include the addition of social links in the profile update #935 [FIX] Remove the email from fields to be updated on users profile. #937 [FIX] Create a different endpoint to handle change of emails for users. #938

[FIX] [ANCHOR: BE PYTHON] Create a different endpoint to handle change of emails for users. #912 [FIX] [ANCHOR: BE PYTHON] Remove the email from fields to be updated on users profile. #914 [FIX] [ANCHOR: BE PYTHON] Include the addition of social links in the profile update #915

Motivation and Context

How Has This Been Tested?

This has been tested with pytest and postman.

Screenshots (if appropriate - Postman, etc):

bugfix profile bio

bugfix profile username

bugfix profile fake social link

bugfix profile fileds empty

bugfix profile email sent for verify_email postman

bugfix profile email receieved

bugfix profile recovery email successfully updated

bugfix profile recovery email now added to user profile

Types of changes

Checklist:

johnson-oragui commented 3 weeks ago

@trevorjob

kindly review