[2주차/여니] 워크북 제출합니다

[zeoueon]

✅ 워크북 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

[YoungJJun]

2주차 피드백

1. 회원가입
   • POST 메서드 적절하고 , URI 또한 RESTful 해요!
2. 홈 화면 조회
   • 페이징을 추가한다면 어떻게 구현할지? 생각해보면 좋을 것 같습니다.
   • 멤버ID를 내려줄 필요가 있을까? 라는 생각을 했는데 이유가 있다면 문제는 없을 것 같아요.
   • regionName=안암동 “” 제거해도 될 것 같아요.
   • 추가로 애초에 지역을 검색해서 백엔드로 넘겨주는 구조가 아니고 제시된 지역들 중 선택하는 방식이니까
     선택 가능한 지역 리스트들을 보여주려면 결국 별도의 API를 통해 존재하는 지역을 조회해야 할 것 같아요.
     그러면 지역 이름에 대한 ID까지 같이 조회해서 던져놓으면 해당 요청 자체도 문자열이 아닌 ID값으로 할 수 있을 것 같아요.
     ?regionId=1 이런식으로요! 그러면 일단 성능면에서는 유리할 것 같습니다.
3. 마이페이지 리뷰 작성
   • Path Variable이 멤버미션이니 member-missions/ 으로 시작하도록 통일하거나 mission/{mission-id}/reviews로 가는게 좋을 것 같아요. 후자를 선택하게 되면 미션ID만 이용해 바로 리뷰를 추가하는게 아닌 해당 멤버미션을 찾아서 검증하는 절차가 필요할 것 같구요.
4. 미션 목록 조회 (진행 중, 완료)
   • missionList가 리스트라서 [ ] 에 넣어야 할 것 같아요.
   • Query Param에 “” 제거
   • endpoint 관련해서 멤버미션 테이블에 대한 조회로 봐야할 것 같기도 해요. 이건 개인적인 생각입니다. 멤버미션이 맞다 라기 보다는 GET missions는 미션 리스트 조회의 느낌이 있어서 내 미션목록과 관련된 내용은 멤버미션이 적절한거 같은데? 라는 생각이 들었어요.
5. 미션 성공 누르기
   • PATCH로 상태변경 좋아요~

---

여니 2주차 수고하셨어요. 꼭 필요한 부분들은 수정해주시고 개인적으로 궁금한거 공부해주시면 좋을 것 같습니다. (질문도 환영입니다) 다음주도 화이팅 해주세요.