[REFACTOR] 서브태스크 관련 스키마 변경사항 반영#262
Conversation
- 서브태스크 완료 상태 변경 API에 추가된 필수 date 파라미터를 반영했습니다 - 서브태스크 수정 요청에서 제거된 completed 필드를 반영했습니다
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (5)
💤 Files with no reviewable changes (1)
Walkthrough서브태스크 상태 변경 요청에 날짜 컨텍스트를 추가하고, 상세 서브태스크 수정 요청에서 Changes서브태스크 API 스키마 변경
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Timo Performance ReportBundle Size — timo-web
Lighthouse — timo-web
Image Optimization — timo-web
측정 커밋: |
ISSUE 🔗
close #261
What is this PR? 🔍
백엔드에서 서브태스크 관련 스키마가 변경되어(완료 상태 변경 API에 date 파라미터 필수화, 수정 요청에서 completed 필드 제거) 클라이언트를 반영했습니다.
배경
changeSubtaskStatus) API가todoId,subtaskId만으로 대상을 특정했습니다.todoId를 여러 날짜 인스턴스가 공유하기 때문에, 날짜 지정 없이는 완료 상태가 어느 날짜 인스턴스에 반영되는지 특정할 수 없었습니다. (이전에startTimer([FIX] 반복 투두를 오늘이 아닌 날짜에서 재생해도 해당 날짜에 귀속되도록 수정 #254) 등 다른 타이머/투두 API에도 동일한 이유로 date 파라미터가 추가된 적이 있습니다.)changeSubtaskStatus에 필수date쿼리 파라미터를 추가했고, 클라이언트의 모든 호출부에서 해당 서브태스크가 속한 투두의 날짜를 함께 전송하도록 반영했습니다.API 클라이언트 (generated)
changeSubtaskStatus가ChangeSubtaskStatusParams({ date: string })를 필수로 받도록 타입과 mutation 훅을 갱신하고,TodoSubtaskUpdateRequest에서completed필드를 제거했습니다.https://api.timo.kr/v3/api-docs)에 맞춰 반영했습니다.pnpm gen:api로 재생성한 뒤, 이번 이슈와 무관한 캘린더/온보딩 등 다른 도메인의 재생성 diff는 제외하고todo.ts의changeSubtaskStatus관련 변경만 수동으로 반영했습니다.서브태스크 완료 상태 변경 호출부
changeSubtaskStatus호출에params: { date }를 추가했습니다.date파라미터 없이는 타입 에러가 발생하고, 반복 TODO에서 완료 상태가 잘못된 날짜 인스턴스에 반영될 수 있습니다.dateKey,focusView.date, 상세 모달의date인자)을 그대로 전달했습니다.상세 모달 수정 요청 빌더
buildDetailTodoSubtasksUpdateRequest에서completed필드 전송을 제거했습니다.TodoSubtaskUpdateRequest에서completed필드가 제거되어 더 이상 요청 body에 포함될 수 없습니다.To Reviewers
이번 PR 범위에는 포함하지 않았지만,
pnpm gen:api재생성 과정에서TodoUpdateRequest.memo필드가 사라지고 별도의 memo 전용 엔드포인트가 새로 생긴 것을 확인했습니다. 이는 별도 이슈로 추후 처리할 예정입니다.Screenshot 📷
Test Checklist ✔
pnpm check-types통과pnpm lint통과