Skip to content

3주차 - 민지/정민지 워크북 과제 제출#7

Merged
munwalk merged 2 commits intoUMC-KWU:minjifrom
minji-jeong516:main
Apr 5, 2026
Merged

3주차 - 민지/정민지 워크북 과제 제출#7
munwalk merged 2 commits intoUMC-KWU:minjifrom
minji-jeong516:main

Conversation

@minji-jeong516
Copy link
Copy Markdown
Collaborator

No description provided.

@minji-jeong516 minji-jeong516 changed the title 3주차 - 민지/정민지 워크북 과제 제 3주차 - 민지/정민지 워크북 과제 제출 Apr 1, 2026
Copy link
Copy Markdown
Collaborator

@munwalk munwalk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3주차 과제 수고하셨습니다!
API 설계를 보니 /api/v1/...로 버전 관리를 하신 점이 좋습니다! 홈 화면을 상단과 하단으로 분리해서 각각의 목적에 맞게 API를 구성하신 점도 훌륭하네요. GET /api/v1/missions/challenging, GET /api/v1/missions/completed로 진행중/완료 미션을 명확히 나눈 것도 직관적입니다.
페이징 처리도 page, size로 구현하셨고, Authorization 헤더를 일관되게 사용하신 점이 좋습니다. PATCH /api/v1/missions/{memberMissionId}/success에서 Path Variable을 잘 활용하셨네요!
다만, 리뷰 작성 API에서 Path Variable 설명에는 memberMissionId가 있는데, 실제 Endpoint /api/v1/reviews에는 Path Variable이 없습니다. memberMissionId를 Path Variable로 사용하려면 /api/v1/member-missions/{memberMissionId}/reviews 형태로 수정하고, Request Body에서는 제거하는 게 RESTful합니다! 그리고 Response Body 예시가 있으면 프론트엔드 입장에서 더 명확하게 이해할 수 있을 것 같아요!
수고하셨습니다!

@munwalk munwalk merged commit 8eaf35a into UMC-KWU:minji Apr 5, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants