Skip to content

[0,1주차/유리] 워크북 제출합니다#17

Open
Kohseoyoung wants to merge 4 commits intoUMC-Inha:yuri/mainfrom
Kohseoyoung:main
Open

[0,1주차/유리] 워크북 제출합니다#17
Kohseoyoung wants to merge 4 commits intoUMC-Inha:yuri/mainfrom
Kohseoyoung:main

Conversation

@Kohseoyoung
Copy link

✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?(중요)
  • 적절한 라벨을 설정하였나요?
  • 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

@YoungJJun
Copy link
Collaborator

0주차 피드백

유리 mission에 사진을 첨부하실 때 실제 사진 파일을 해당 .md 파일이 포함된 폴더에 넣고 해당 사진을 끌어서 작성중인 파일로 가져오면 사진을 첨부할 수 있어요. 노션에서 작성 후 복사하는 과정에서 사진은 복사가 따로 안되어서 안보이네요ㅠㅠ 우선 저는 노션 확인이 가능해서 노션보고 리뷰할게요. 다음부터는 반영해주세요!


  • 물리값 논리값 헷갈리지 않고 잘 구분해주셨어요.

  • 멤버에 status를 통해 비활성까지 고려한 점 너무 좋아요.

  • 약관 테이블 필수/선택이 is_optional 인데 필수0, 선택1 이라고 작성해주셨는데 완전히 개인적 의견이지만 컬럼이 is_optional 이고 1을 일반적으로 true라고 생각해서 1일경우 옵션 즉 선택이 어울리는 것 같다는 생각을 했습니다.

  • 리뷰에 별점이 float로 설정되어 있습니다. 0.5단위니까 문제 없을수도 있으나 부동소수점 보다는 DECIMAL(2, 1)과 별점에 x10 해서 int로 저장하는 방법도 있어요. 참고해주세요.

  • 나이는 직접 저장하게 되면 매년 수정이 필요한데 생일을 저장하면 바로바로 계산이 가능해요.

  • 사용자 약관에 동의여부가 bigint로 잘못 설정하신 것 같아요.

  • 포인트 이력 테이블이나 가게 운영시간 테이블 (JSON 타입의 컬럼도 가능) 등 추가해볼 수 있을 것 같아요.

유리 0주차 수고하셨습니다.!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chapter01_DB와 친해지기 - SQL문 Chapter00_DB와 친해지기 - ERD 설계

2 participants