Skip to content

[fix] #207 - 메모 존재 여부 반영하도록 dto 및 로직 수정 - #208

Merged
laura-jung merged 1 commit into
developfrom
fix/#207-today-todo-response
Sep 26, 2026
Merged

laura-jung merged 1 commit into
developfrom
fix/#207-today-todo-response

Conversation

@laura-jung

@laura-jung laura-jung commented Sep 26, 2026 •

Copy link
Copy Markdown
Collaborator

관련 이슈 🛠

작업 내용 요약 ✏️

기존 todo 조회에서 명세서 속 내용인 hasMemo가 아닌 hasSubtask가 전달되는 오류가 있어 수정하였습니다.

주요 변경 사항 🛠️

  • hasSubtask 삭제
  • hasMemo 추가 및 로직 추가(HomeTodoMapper)

트러블 슈팅 ⚽️

Summary by CodeRabbit

  • 버그 수정
    • 홈 및 오늘 할 일 목록에서 표시하는 정보가 하위 할 일 유무에서 메모 유무로 변경되었습니다.
    • 메모가 비어 있거나 공백만 있는 경우에는 메모가 없는 것으로 표시됩니다.

@laura-jung laura-jung self-assigned this Sep 26, 2026
@laura-jung laura-jung linked an issue Sep 26, 2026 that may be closed by this pull request
4 tasks
@coderabbitai

coderabbitai Bot commented Sep 26, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: Team-Timo/Timo-Server/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 60dd83fc-4573-4288-bb5e-33f3b70c3b7a

📥 Commits

Reviewing files that changed from the base of the PR and between 99218bf and 6d61002.

📒 Files selected for processing (3)
  • src/main/java/com/Timo/Timo/domain/home/dto/response/HomeResponse.java
  • src/main/java/com/Timo/Timo/domain/home/dto/response/TodayResponse.java
  • src/main/java/com/Timo/Timo/domain/home/mapper/HomeTodoMapper.java
 _________________________________________________________________
< I have no memory of this place...but I'll find the bugs anyway. >
 -----------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@laura-jung
laura-jung merged commit d9b56b3 into develop Sep 26, 2026
1 of 2 checks passed
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] 오늘 TODO 조회 응답을 명세에 맞게 수정

1 participant