[4주차/제이] 워크북 제출합니다.

[yeongjun25]

✅ 실습 체크리스트

• 이론 학습을 완료하셨나요?
• 미션 요구사항을 모두 이해하셨나요?
• 실습을 수행하기 위한 공부를 완료하셨나요?
• 실습 요구사항을 모두 완료하셨나요?

✅ 컨벤션 체크리스트

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

📌 주안점

[Sangrix]

제이 시험기간인데 고생 많았어요!!

[YoungJJun]

4주차 피드백

• yaml 파일 환경변수 설정하신 점 좋습니다.

• ignore 에 .env 추가도 좋아요.

• build.gradle 파일 의존성 설정 각자 상황에 맞게 잘 수정해주셨습니다.

• 워크북 내용에는 store가 분리되지 않았던 것 같은데 각자 설계에 맞게 분리해주신 점 좋아요.
  그런데 store 하위에 있는 Region은 멤버에는 쓰이지 않는걸까요?? 멤버도 관심지역을 Region과의 매핑을 통해 갖게된다면 Region 또한 분리하는게 자연스러울 수 있습니다.

• 멤버 하위로 Food가 있습니다. 아마 중식당, 일식당 등 음식 카테고리를 의미하는 것 같은데 멤버도 Food를 갖지만 가게도 대표 카테고리를 갖을 수 있다면 Food 또한 별도의 도메인으로 두는게 좋을 수 있습니다.

• 멤버 하위로 약관을 두는것도 사용자가 약관을 동의한다는 부분에서 어느정도 이해되지만 추후 범용적으로 사용될 여지가 있으면 분리 고려해주세요.

(최신 DB 설계를 확인할 수 없고, 답이 있는 문제는 아니라 분리해라 마라 정확하게 말씀드리기는 어려울 것 같습니다. 한 번 설계를 확인해보시고 분리를 고려해보시면 될 것 같아요)

---

피드백에 대한 수정은 필수가 아니지만 DB 상황 보시고 필요한 경우 DB 설계를 바꾸거나 코드를 바꾸거나 해주시면 좋을 것 같아요!

이대로 진행하다 나중에 문제가 생기면 해결하기 더 어려울 것 같아서요.

수정은 5주차 PR전에만 진행 후 머지해주시면 됩니다.

제이 4주차 수고하셨습니다! 시험 잘보세요~ 🍎