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

[tjdals-dbs]

✅ 워크북 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

[YoungJJun]

2주차 피드백

1. 홈화면
   • home이 RESTful하지는 않지만 많이 사용하는걸로 알고있어요.
   • userId는 토큰에 포함되기에 포함하지 않아도 됩니다.
     추가로 토큰에 포함되어서 필요하지 않다는 점을 제외해도 userId에 다른 값을 대입해서 다른 회원의 홈화면을 조회할 수 있다는 위험성도 있어요.
   • 토큰 사용에 따라 Path Variable에는 userId 제외
   • isSuccess, code, message등 팀원분들 응답 한 번 확인해보시면 결과외에 전달하는 Response에 포함되는 값이 무엇이 있는지 확인해보면 좋을 것 같아요.
   • 페이징을 적용한다면 어떤게 추가될지 생각해보시면 좋을 것 같아요.
2. 마이페이지 리뷰 작성
   • 별점에 대해 *10 값 저장하는 방식은 좋아요! 추가로 프로젝트에서 명세서 작성하면 이 점을 명시하면 더 좋을 것 같아요.
   • 홈화면과 동일하게 응답통일! 전체적으로 진행하면 좋을 것 같아요. (isSuccess, code, …) 이후로는 계속 리뷰가 중복되니까 더 이상 똑같은 얘기는 안넣을게용. 3,4,5에 대해서도 해주시면 됩니다.
3. 미션 목록 조회
   • GET /api/member-missions?status=COMPLETED 등으로 endpoint 수정을 고려해주세요. userId삭제하게 되면 /api/missions가 될 것 같은데 이거는 미션 리스트 조회에 더 가깝다고 생각해서 알려드려요.
   • Query Param 수정하신다고 하셨던 것 같아요. 완료랑 진행중이 탭이 나뉘어져 있고 둘 중 하나만 조회하는게 맞기에 Query Param수정 및 응답에서 status가 불필요할 것 같아요.
4. 미션 성공 누르기
   • PATCH 사용 좋습니다.
   • 수정하는 대상이 멤버미션의 status이기 때문에 endpoint를 멤버미션에 맞춰서 수정해야 적절할 것 같아요.
5. 회원가입
   • URI 아주 RESTful해요!
   • Req Body에 “name” : “홍길동” 꼴로 수정해야 할 것 같아요 등호가 있어요 지금
   • 정보 노출까지 고려해서 설계하신점 좋아요! 프로젝트에 따라 다른 데이터도 전달해도 됩니다.

---

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