okuda-seminar / Twitter-Clone

0 stars 0 forks source link

[go/users] Integrate UpdateUsername and UpdateBio into UpdateProfile #307

Closed ryuju0911 closed 4 months ago

ryuju0911 commented 5 months ago

Summary

Integrate UpdateUsername and UpdateBio into UpdateProfile.

Deadline

6/28

References

None

Notes

None

ryuji0123 commented 5 months ago

Do we plan to create API URI for each prop of users like name, location?

ryuju0911 commented 5 months ago

@ryuji0123 I think managing the API in this manner will become difficult. Would it be a better idea to parse the payload and update its parameters instead?

ryuji0123 commented 5 months ago

I guess one way is making client code responsible for fulfilling only updated props. This way, server code shouldn't worry about if passed payload should be fully used to update DB data or check diffs first.

ryuju0911 commented 5 months ago

I see. I will adopt your idea.