Skip to content

[fix] #237 - 투두 삭제 API의 date 파라미터를 임시로 선택값으로 변경 - #238

Merged
aneykrap merged 2 commits into
developfrom
fix/#237-optional-delete-date
Oct 15, 2026
Merged

aneykrap merged 2 commits into
developfrom
fix/#237-optional-delete-date

Conversation

@aneykrap

Copy link
Copy Markdown
Collaborator

관련 이슈 🛠

작업 내용 요약 ✏️

요청 결과
date 없음 기존처럼 투두 전체 삭제 (반복 여부와 상관없음)
date 있음 #224 동작 그대로 (반복 투두는 date를 포함한 이후 일정만 삭제)
date 형식 오류, 빈 값 (date=) TODO_400

주요 변경 사항 🛠️

  • [Todo] : TodoService.deleteTodo에서 date가 없으면 파싱하지 않고 removeTodo로 전체 삭제
  • [Todo] : TodoControllerDocs 삭제 API에 date 미전달 시 동작 설명 추가, 400 응답 설명에서 date 누락 문구 제거
    • 웹이 Swagger 명세로 API 코드와 Zod 스키마를 자동 생성하고 있어서 date의 required = true는 그대로 두었습니다. 웹은 계속 date를 필수로 보내도록 개발하면 됩니다.

트러블 슈팅 ⚽️

  • date 필수 상태의 #224가 develop에 머지되어, 웹 배포 전에 다른 기능을 배포하면 기존 웹의 삭제가 실패하는 문제가 있었습니다.
  • #224를 revert하면 나중에 다시 넣을 때 revert를 다시 revert해야 하고 그 사이 쌓인 변경과 충돌할 수 있어서, date를 잠시 선택값으로 받는 방식을 택했습니다.

테스트 결과 📄

  • ./gradlew build 성공

리뷰 요구사항 📢

  • 이 PR이 develop에 머지되기 전까지는 deploy 머지를 멈춰주세요.
  • date를 선택값으로 받는 건 임시입니다. 웹이 배포를 요청하면 date를 다시 필수로 되돌려 서버를 먼저 배포하고, 웹이 1~2시간 이내에 따라 배포할 예정입니다. ([fix] 투두 삭제 API의 date 파라미터를 선택값으로 변경 #237 Todo에 남겨두었습니다)

@coderabbitai

coderabbitai Bot commented Oct 15, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: Team-Timo/Timo-Server/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6b794283-5ab4-43aa-9281-1e07d9bd4126
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[fix] 투두 삭제 API의 date 파라미터를 선택값으로 변경

1 participant