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/update ddl #53

Closed seungholee-dev closed 1 year ago

seungholee-dev commented 1 year ago

해당 PR은 구현된 Create Battle과 관련된 배틀, 라운드, 팀 생성/시작/종료에 관한 PR을 날리기 전 DDL을 수정한 PR입니다. 주요 수정내용은 다음과 같습니다.

Approve되면 Production RDS에 해당 내용 변경예정입니다. 추가로, Create Battle관련 PR도 수정된 RDS에서 동작 최종 확인 후 PR 날리겠습니다.

Jaewook-Lee commented 1 year ago
  1. Round 테이블의 startTime 을 NULL로 변경한 이유는 무엇인가요?
  2. Dummy Data를 삽입할 때 battleId를 "000002" 를 사용할 것을 제안합니다. 이미 "000001"이 있기 때문에 같은 battleId에 팀을 추가하면 하나의 방에 팀이 총 4개가 됩니다. 한 방에 팀이 2개여야 하는 점을 생각해서 battleId를 바꿔서 예시 데이터를 삽입하면 혼동이 줄어들 것 같습니다.
  3. DDL이 backend directory에 1개, setup directory에 1개 있습니다. 둘이 동일한 DDL이면 둘 중 하나는 지워도 될 것 같습니다.
seungholee-dev commented 1 year ago
  1. startTime을 NULL로 변경하게 된 이유는 저번 회의에서처럼 배틀은 시작하였지만 라운드를 진행하지 않았을 경우를 고려하여 적용하였습니다. 방생성과 동시에 라운드 생성과 팀 생성을 하게되는데 이때, 2 라운드나 3라운드의 경우는 생성과 동시에 시작하지 않는 경우에 해당해서 적용해보았습니다. 혹시 어떻게 생각하시나요?

  2. 넵 000002도 상관 없을 것 같습니다. 로컬테스트용이긴하지만 000002로의 수정 좋은 것 같습니다.

  3. 정말이네요! setup directory 기준이 최신으로 보이고 재욱님도 이것으로 적용하신 것 같아서 backend directory에 있는 model.sql하나 지우겠습니다!

seungholee-dev commented 1 year ago

우선, 2번 3번 변경사항 적용했습니다.

Jaewook-Lee commented 1 year ago

아 맞네요. 1번 사항 이해했습니다.

Binsk-dev commented 1 year ago

startTime 필드를 NULL로 바꿨는데, 그럼 저번 회의에서 얘기한 방 생성과 동시에 Round 테이블을 만들때, 시작 시간을 미리 계산해서 넣어주는 방식은 사용하지 않기로 결정한건가요?

seungholee-dev commented 1 year ago

음 고민되는 것이 배틀 중간 과정에서 방송 진행자가 배틀결과를 바탕으로 시청자들과 대화하는 화면이 있는 것으로 알고 있는데 시작 시간이 정해지지 않을 것 같아서 넣으려면 조금 고민이 되더라구요 어떻게 생각하시나요 :thinking:

Jaewook-Lee commented 1 year ago

흠.. 라운드 시작 요청할 때마다 요청 시각을 Round table에 넣는 방법은 어떤가요?

seungholee-dev commented 1 year ago

우선 현재로서는 재욱님이 말씀하신대로 startRound Action을 websocket으로 요청할 때마다 Round Table의 해당 라운드의 시작 시간을 업데이트하는 방식으로 구현 완료했습니다.

seungholee-dev commented 1 year ago

넵 머지진행하겠습니다!