kookmin-sw / capstone-2023-05

capstone-2023-05 created by GitHub Classroom
https://capstone-2023-05.vercel.app
0 stars 0 forks source link

refactor: init-join-handler #75

Closed Jaewook-Lee closed 1 year ago

Jaewook-Lee commented 1 year ago

요청 내용

like 액션 요청할 때 round 값을 넣지 않고 싶어요!

요청 사항을 만족시키기 위해 DynamoDB에 현재 사용자가 속한 라운드 값도 같이 저장하기로 정했습니다.
따라서 initJoin 액션을 요청할 때부터 DynamoDB에 currRound 값으로 빈 스트링 ""을 초기 값으로 부여하는 방향으로 수정했습니다.

initJoinResult에서 image 필드도 같이 받아오고 싶어요!

이제 initJoinResult 에서 새로운 field "image"를 받을 수 있습니다.

테스트 방법

  1. 임의의 유저 한 명이 initJoin 요청을 수행
  2. DynamoDB에서 currRound 값에 "" 가 부여됐는지 확인
  3. initJoinResult 에서 새로운 field 값 "image"가 오는지 확인
Jaewook-Lee commented 1 year ago

네네 currRound 맞아요!

Binsk-dev commented 1 year ago

그런데 생각해보면 사실 모든 유저가 같은 라운드에 있을 거니깐 유저별로 라운드 정보를 갖고 있는 것 비효율적인 것 같습니다. 따라서 유저 정보 바깥에 별도로 정보를 갖고 있는게 좋아보이기는 하는데 당장 구현이 어렵다면 일단 이렇게 가는게 좋을 것 같아요.

seungholee-dev commented 1 year ago

확인했어요! 감사해요!