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

[Kohseoyoung]

✅ 워크북 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

[YoungJJun]

0주차 피드백

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

---

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

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

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

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

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

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

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

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

[YoungJJun]

1주차 피드백

• 머지하시기 전에 피어리뷰 추가 후 진행해주세요!
• mission.md에는 쿼리문이 있는데 노션 워크북에는 설명만 있어요! PR 링크를 제출하면 문제 없지만 유리 제가알기로 노션 링크 제출하신걸로 알고 있어서 PR 내용 복사해서 노션에도 채워주셔야 합니다. -> 2주차 기간까지 제출되지 않으면 이 부분도 벌점 부여되요 ㅠㅠ 꼭 수정해주세요.

---

1번 쿼리 잘 작성해주셨어요!

2번 쿼리도 깔끔합니다!

3번
진행중과 진행완료에 대해 동시에 조회하는 쿼리로 작성해주셨어요. 해당 화면에서는 진행중/진행완료를 분리해서 보여줘야 하기 때문에 어떤탭이 선택되었는지에 따라 둘 중 하나로만 조회해야 할 것 같아요.

mm.status 를 가져오는건 아마 사진상에 미션마다 성공 여부가 적혀있기 때문인것 같아요. 근데 결국 진행중 탭에는 다 진행중으로 표시될거고 완료 탭에서는 다 성공이니까 꼭 가져올 필요는 없을것 같아요!

4번
미션에 대해 시도와 완료횟수를 카운트 하셨는데 이 부분은 요구사항이랑 다른 것 같아요. 해당 부분은 10개 달성마다 포인트를 지급하는게 목적인것 같아요.
그래서 해당 멤버가 해당 지역에서 성공한 미션의 수를 count 하고 10개마다 보상을 받았을테니 mod 10 해주시면 9까지 count 되고 10 달성하는 경우 보상 주어지고 다시 0으로 바뀌게 됩니다.

MYMISSION 부분에 대해서는
필요한데 SELECT되고 있지 않은 컬럼들이 있어보여요. 가게이름, 가게의 카테고리(category나 food로 구현하셨던거 같아요) 이 부분 추가로 조회가 필요할 것 같아요.

---

유리 1주차도 수고하셨습니다. 피어리뷰, 노션에 내용추가 후 머지해주세요!
2주차도 화이팅입니다~