Skip to content

isMine 응답 필드 값 위치 변경#255

Merged
kelly6bf merged 1 commit intodevfrom
feature/#251-is_mine_response_value_add
Feb 4, 2026
Merged

isMine 응답 필드 값 위치 변경#255
kelly6bf merged 1 commit intodevfrom
feature/#251-is_mine_response_value_add

Conversation

@kelly6bf
Copy link

@kelly6bf kelly6bf commented Feb 4, 2026

😉 연관 이슈

Resolves #251

🧑‍💻 수행 작업

  • isMine 응답 필드 값 위치 변경

📢 참고 사항

X

Summary by CodeRabbit

변경 사항

  • 리팩토링
    • 일일 할일 이력 조회 API의 응답 구조를 개선했습니다.
    • 소유권 정보를 개별 항목 수준에서 응답 최상위 수준으로 이동하여 데이터 구조를 단순화했습니다.
    • 데이터 전송 객체들을 재정렬하여 API의 일관성을 높였습니다.

@coderabbitai
Copy link

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

isMine 필드가 TodoHistoryDto(개별 투두 히스토리 수준)에서 FindTargetMemberTodayTodoHistoriesDto(최상위 응답 수준)로 이동되었습니다. DailyTodoHistory 엔티티에서 isMine 메서드도 제거되었고, 이에 맞게 DTOs, 서비스, API 응답 및 테스트가 업데이트되었습니다.

Changes

Cohort / File(s) Summary
API 응답 모델 업데이트
src/main/java/site/dogether/dailytodo/controller/v2/dto/response/GetChallengeGroupMemberTodayTodoHistoryApiResponseV2.java
외부 record에 isMine 필드를 추가하고, 내부 TodoData record에서 제거했습니다. from() 팩토리 메서드도 함께 업데이트되었습니다.
DTO 구조 변경
src/main/java/site/dogether/dailytodohistory/service/dto/FindTargetMemberTodayTodoHistoriesDto.java, src/main/java/site/dogether/dailytodohistory/service/dto/TodoHistoryDto.java
FindTargetMemberTodayTodoHistoriesDto에 isMine 필드를 첫 번째 컴포넌트로 추가하고, TodoHistoryDto에서 isMine 필드를 제거했습니다.
엔티티 및 서비스 레이어
src/main/java/site/dogether/dailytodohistory/entity/DailyTodoHistory.java, src/main/java/site/dogether/dailytodohistory/service/DailyTodoHistoryService.java
DailyTodoHistory에서 isMine 메서드를 제거했고, DailyTodoHistoryService에서 DTO 생성 시 새로운 필드 순서에 맞게 업데이트했습니다.
테스트 업데이트
src/test/java/site/dogether/docs/dailytodo/v1/DailyTodoControllerV1DocsTest.java, src/test/java/site/dogether/docs/dailytodo/v2/DailyTodoControllerV2DocsTest.java
DTO 생성자 호출을 새로운 필드 구조에 맞게 수정하여 테스트 데이터를 업데이트했습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • dogether-backend#252: 동일한 DTOs 및 서비스 메서드들을 수정하지만 반대 방향의 변경이 포함되어 있으므로 관련성이 있습니다.

Suggested labels

deploy

Suggested reviewers

  • Hwangseoeun
🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목 'isMine 응답 필드 값 위치 변경'은 주요 변경사항을 명확하게 설명하며 간결하고 구체적입니다.
Description check ✅ Passed PR 설명이 템플릿 구조를 따르고 연관 이슈, 수행 작업, 참고 사항을 완벽하게 포함하고 있습니다.
Linked Issues check ✅ Passed 코드 변경이 이슈 #251의 요구사항인 'isMine 응답 필드 추가'를 충분히 구현하고 있습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항은 isMine 필드의 위치 변경과 관련된 범위 내 변경이며, 무관한 수정사항이 없습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/#251-is_mine_response_value_add

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


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.

❤️ Share

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

@kelly6bf kelly6bf merged commit 7312eca into dev Feb 4, 2026
7 of 8 checks passed
@kelly6bf kelly6bf added the deploy 해당 라벨을 붙이면 배포 작업을 진행합니다. (붙이지 않으면 배포 작업이 진행되지 않고 머지만 됩니다.) label Feb 4, 2026
@kelly6bf kelly6bf deleted the feature/#251-is_mine_response_value_add branch February 4, 2026 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy 해당 라벨을 붙이면 배포 작업을 진행합니다. (붙이지 않으면 배포 작업이 진행되지 않고 머지만 됩니다.)

Projects

None yet

1 participant