Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable Users to update their email #449

Closed
4 tasks
eleanorreem opened this issue Jun 4, 2024 · 2 comments
Closed
4 tasks

Enable Users to update their email #449

eleanorreem opened this issue Jun 4, 2024 · 2 comments
Assignees
Labels
complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days feature/enhancement New feature or request priority: now Should be prioritized immediately. state: approved Ready to go. Not blocked or pending.
Milestone

Comments

@eleanorreem
Copy link
Contributor

Overview

Currently our PATCH /user endpoint only allows the user to update their name. We want the user to be able to update their email as well.

Action Items

  • Take a look at the PATCH /user endpoint in /bloom-backend/src/user/user.controller.ts
  • Update the UpdateUserDTO to contain a email property. This should not be required.
  • If a user decides to update their email, this will need to be changed in Firebase as well as in our database. Therefore the updateUser service method needs to handle this. If the user supplies a new email in the DTO, the method should first update the email in Firebase and then when that passes, it should update the user in the database. If the user email update fails, it should throw with the appropriate errors such as the ones in the createUser method.
  • Create a service test to safeguard updating an email. There should be a pass and fail scenario.

Resources

@eleanorreem eleanorreem added feature/enhancement New feature or request complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days priority: now Should be prioritized immediately. state: approved Ready to go. Not blocked or pending. labels Jun 4, 2024
@kyleecodes kyleecodes added this to the 02. Roadmaps milestone Jun 6, 2024
@kyleecodes kyleecodes moved this to To-Do Bloom Backend in Chayn Tech Volunteer Advancement Jun 9, 2024
@eleanorreem eleanorreem self-assigned this Jun 11, 2024
Copy link

Thank you @eleanorreem you have been assigned this issue!
Please follow the directions in our Contributing Guide. We look forward to reviewing your pull request shortly ✨


Support Chayn's mission? ⭐ Please star this repo to help us find more contributors like you!
Learn more about Chayn here and explore our projects. 🌸

@eleanorreem
Copy link
Contributor Author

@kyleecodes I have to fastforward this issue so i will pick it up.

@kyleecodes kyleecodes moved this from To-Do Bloom Backend to In Progress in Chayn Tech Volunteer Advancement Jun 12, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Chayn Tech Volunteer Advancement Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days feature/enhancement New feature or request priority: now Should be prioritized immediately. state: approved Ready to go. Not blocked or pending.
Projects
None yet
Development

No branches or pull requests

2 participants