HackYourFuture-CPH / fp-class18

Interior design webshop
MIT License
1 stars 0 forks source link

Backend: Endpoint to update delivery info for user - PUT api/users/{user_id} #45

Closed Jul-S closed 2 years ago

Jul-S commented 2 years ago

User story

Who: As a developer What: I want to be able to update delivery info for user Why: so that we can use form to update delivery info

Acceptance criteria

The endpoint needs to return a result that allows user to update delivery info through the form marked red below: image

Implementation details

Check the README file here https://github.com/HackYourFuture-CPH/fp-class18/#-working-with-api-documentation---swagger and this article should also help you with that https://blog.logrocket.com/documenting-your-express-api-with-swagger/

semih1239 commented 2 years ago

Hi @Jul-S . I think that issue need to be PUT method. Because the purpose of this part is to update the user address. from database

Jul-S commented 2 years ago

Hey @semih1239 i think we need POST user anyway to add new user to our DB, maybe you can create it

semih1239 commented 2 years ago

Hey @semih1239 i think we need POST user anyway to add new user to our DB, maybe you can create it

Yes @Jul-S . You are right. I create an issue for it