Skip to content

참여중인 특정 챌린지 그룹에 속한 특정 그룹원의 당일 데일리 투두 히스토리 전체 조회 api 명세 문서 응답 필드 내용 수정#265

Merged
Hwangseoeun merged 1 commit into
devfrom
feat/#264-find_all_history_api_docs_fix
Apr 20, 2026
Merged

참여중인 특정 챌린지 그룹에 속한 특정 그룹원의 당일 데일리 투두 히스토리 전체 조회 api 명세 문서 응답 필드 내용 수정#265
Hwangseoeun merged 1 commit into
devfrom
feat/#264-find_all_history_api_docs_fix

Conversation

@Hwangseoeun

@Hwangseoeun Hwangseoeun commented Apr 20, 2026

Copy link
Copy Markdown
Member

😉 연관 이슈

Resolves #264

🧑‍💻 수행 작업

  • 문서 내 응답 필드 내용 수정

📢 참고 사항

X

Summary by CodeRabbit

릴리스 노트

  • 문서 개선
    • API 응답 필드 설명을 정확하게 수정했습니다. 특정 ID 필드의 설명이 더 정확하게 업데이트되었습니다.

@Hwangseoeun Hwangseoeun self-assigned this Apr 20, 2026
@Hwangseoeun Hwangseoeun added the deploy 해당 라벨을 붙이면 배포 작업을 진행합니다. (붙이지 않으면 배포 작업이 진행되지 않고 머지만 됩니다.) label Apr 20, 2026
@coderabbitai

coderabbitai Bot commented Apr 20, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: c876e8a9-adcf-4186-a0ca-d29ee2eed933

📥 Commits

Reviewing files that changed from the base of the PR and between 6e70f3f and cb08173.

📒 Files selected for processing (1)
  • src/test/java/site/dogether/docs/dailytodo/v1/DailyTodoControllerV1DocsTest.java

📝 Walkthrough

Walkthrough

REST Docs 테스트 파일의 응답 필드 설명을 수정했습니다. getChallengeGroupMemberTodayTodoHistoryV1() 메서드에서 data.todos[].id 필드의 설명을 "daily todo id"에서 "todo history id"로 변경했으며, 다른 테스트 로직이나 구조에는 변경이 없습니다.

Changes

Cohort / File(s) Summary
REST Docs 필드 설명 수정
src/test/java/site/dogether/docs/dailytodo/v1/DailyTodoControllerV1DocsTest.java
응답 필드 data.todos[].id의 설명을 "daily todo id"에서 "todo history id"로 수정했습니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 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 제목이 변경 사항의 주요 내용을 명확하게 설명하고 있습니다. API 명세 문서의 응답 필드 내용 수정이라는 핵심 변경 사항을 적절하게 반영하고 있습니다.
Description check ✅ Passed PR 설명이 저장소의 템플릿을 완벽하게 따르고 있으며, 연관 이슈(#264), 수행 작업, 참고 사항이 모두 명시되어 있습니다.
Linked Issues check ✅ Passed 코드 변경이 연관된 이슈 #264의 요구 사항을 완벽하게 충족하고 있습니다. 응답 필드 설명이 '투두 id'에서 '투두 히스토리 id'로 정확하게 수정되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항이 이슈 #264의 범위 내에 있습니다. 응답 필드 설명 수정 외의 추가적인 변경 사항이 없으므로 범위를 벗어나지 않습니다.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/#264-find_all_history_api_docs_fix

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.

@Hwangseoeun Hwangseoeun merged commit c3f31e2 into dev Apr 20, 2026
7 of 8 checks passed
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