kookmin-sw / capstone-2023-05

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

Update: DiscussionBattle table spec change #61

Closed Binsk-dev closed 1 year ago

Binsk-dev commented 1 year ago

본 PR은 필요 없는 필드와 추가될 필요가 있는 필드를 업데이트 하였습니다.

변경사항 요약

관련 주의사항 테이블이 생성된 걸 보니깐 PascalCase라던지 camelCase와 같은 네이밍 룰이 적용되지 않은채 테이블이 생성되더군요. ex) Round 테이블 생성시 round로 등록됨, roundNo 필드가 roundno로 등록됨 따라서 앞으로는 INSERT와 같은 쿼리문에서 필드 값에 "필드" 이런 식으로 따옴표로 감싸줘야 합니다. 즉 기존에 작성된 여러분의 코드도 이것 때문에 사이드 이펙트 생겨 작동하지 않을 수도 있다는 의미입니다.

무슨 의미인지 잘 모르겠다면 해당 PR의 코드변경 사항을 참고하면 좋습니다.

Jaewook-Lee commented 1 year ago

여러 곳에서 나타나는데 대표적으로 "User" 테이블을 보면 "nickname" 필드는 따옴표로 감싸지만 profile 필드는 따옴표로 감싸지 않네요.
두 필드 모두 소문자로만 작성됐지만 일부 필드만 따옴표로 감싸졌어요.
혹시 이렇게 작성하신 의도가 따로 있는건가요?

seungholee-dev commented 1 year ago

저두 위의 재욱님의 커멘트에 대한 답변 기다리는 중이요! :)

Binsk-dev commented 1 year ago

따옴표를 안 붙인 이유는 사실 Pascalcase, camelCase를 사용하는 심볼 이름들을 명확히 표기하기 위해서 입니다.(안 붙이면 PostgreSQL이 마음대로 lowercase로 만들어 버리더군요) 그래서 2가지 해당되지 않는 단어들은 따옴표를 붙이지 않았던건데... 지금보면 일관성이 깨지는 것 같긴하네요. 그럼 이 부분만 수정하겠습니다.

Binsk-dev commented 1 year ago

네 수정했으니 검토 부탁드립니다들~

seungholee-dev commented 1 year ago

고생하셨습니당!! Merge 했어요!