[1주차/에반] 워크북 제출합니다

[Sangrix]

✅ 워크북 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

[YoungJJun]

1주차 피드백

1번쿼리 좋습니다! ‘ ~ 다음에 또 올게요’ 뒤에 반점 빠졌어요!

2번

• SELECT 하는 항목중에 created_at은 따로 보여주는 화면이 없는것 같은데 이 경우 ORDER BY 에만 사용하시는 거라면 SELECT 할 필요는 없어요!
• is_completed를 따로 조회하지 않고 is_completed를 where절에 넣어서 ‘진행중’, ‘진행완료’ 에 대해서 따로 불러오는게 나을것 같아요! 각각 나눠서 보여줘야 하는데 지금처럼 완료여부를 가져오면 거기서 또 분리를 해야할 것 같아요.
• 가게이름이랑 미션 내용이 필요한데 없는것 같아요. 반면 m.mission_title은 미션제목 같은데 화면에서 미션 이름이 필요한지 모르겠어요. 내용을 조회하려고 하신걸까요? 한 번 확인해주세요.
• 리뷰 남기기 버튼이 있는데 일단 서비스의 목적이 뭔지 저희는 모르지만 저는 가게에 리뷰를 남기는거라 생각해요. (미션에 대해 남기는게 아니구) 이 경우 가게id를 같이 넘겨주면 이후에 리뷰 남기기 버튼을 눌렀을 때 처리가 편리할 것 같아요.

3번은 휴대폰 번호 추가로 불러오면 좋을 것 같습니다.

4번

• 여기서도 미션 제목이 아니라 미션 내용이 필요한 것 같은데 ERD 설계하실 때 title을 ‘10000원 이상의 식사시’ 이런식으로 설계 하신거라면 지금 코드가 맞아요!

• 가게 유형까지 불러와주면 좋을 것 같아요. (중식당)

• 이미 완료한 미션은 제외할 필요가 있어요!

• 미션 10개 도달마다 포인트 추가로 주는 부분 쿼리가 빠진것 같아요.

---

제가 에반의 ERD를 완벽히 이해한게 아니여서 잘못된 피드백이 있을 수 있는데 한 번 확인해주시고 따로 생각한 부분이 있으시다면 알려주세용!
1주차 수고하셨습니다! 2주차 진행전에 머지 후 진행해주시면 됩니다 화이팅~