Closed eleanorreem closed 2 months ago
Hi I would like to take up this issue, can you assign it to me? :)
Thank you @leoseg 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. 🌸
Hi I created the pull request :) https://github.com/chaynHQ/bloom-backend/pull/567
Overview
Take a read through the umbrella ticket. As a sub task to this ticket we want to phase out the use of
req["user"]
from the session-user.controller.ts. The endpoints that need addressing are POST /session-user, POST /session-user/complete, POST /session-user/incomplete. Note that we haven't followed rest patterns here and hopefully this will be refactored at a later date but not part of this ticket.Action Items
req["user"]
, the service methods will need to be refactored to take thereq["userEntity"]
rather thanreq["user"]
. This means the service methods will be taking userEntity rather thanGetUserDto
as its argument. The types need to be updated in the service method and adjustments will need to be made if they are used elsewhere.