Skip to content

[FIX] 독서 기록 수정 시 타임라인 미리보기에 최신 내용 반영 - #371

Merged
kjhyeon0620 merged 1 commit into
develop-demofrom
fix/#370-record-timeline-preview
Sep 20, 2026
Merged

kjhyeon0620 merged 1 commit into
develop-demofrom
fix/#370-record-timeline-preview

Conversation

@kjhyeon0620

@kjhyeon0620 kjhyeon0620 commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

📄 작업 내용 요약

  • 독서 기록 수정 후 목록·요약 미리보기에 최신 내용이 반영되도록 수정

📎 Issue 번호


✅ 작업 목록

  • 기능 구현
  • 코드 리뷰 반영
  • 테스트 코드 작성
  • 문서 업데이트

📝 기타 참고사항

Summary by CodeRabbit

  • 버그 수정
    • 동일한 서재에 기록이 있으면 타임라인 미리보기가 저장된 문구보다 최신 기록 본문을 우선 표시합니다.
    • 기존 미리보기 문구가 null이거나 공백인 경우에도 기록 내용으로 올바르게 대체됩니다.
    • 기록 내용이 공백인 경우에는 기존과 같이 이미지 개수 기반의 안내 문구가 표시됩니다.

@coderabbitai

coderabbitai Bot commented Sep 20, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: d6ee155a-1871-4eeb-8ebc-f775caf03c46

📥 Commits

Reviewing files that changed from the base of the PR and between dfe649e and 1fc12fd.

📒 Files selected for processing (2)
  • src/main/java/app/nook/timeline/service/TimelineQueryService.java
  • src/test/java/app/nook/timeline/service/TimelineQueryServiceTest.java

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

Record가 존재하면 목록과 요약 미리보기가 기존 timeline 문구 대신 최신 Record 내용을 사용합니다. 최신 본문이 공백이면 이미지 개수 fallback을 사용합니다. 관련 조건과 매개변수 테스트를 추가했습니다.

Changes

타임라인 미리보기 업데이트

Layer / File(s) Summary
최신 기록 미리보기 생성 및 검증
src/main/java/app/nook/timeline/service/TimelineQueryService.java, src/test/java/app/nook/timeline/service/TimelineQueryServiceTest.java
Record 존재 여부를 기준으로 기록 미리보기 경로를 적용합니다. 목록과 요약 미리보기가 최신 Record 본문을 사용하고, 본문이 공백이면 "3개의 이미지"를 반환하는지 검증합니다.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix · Severity of issue fixed: Medium

🚥 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%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 독서 기록 수정 후 타임라인 미리보기에 최신 내용을 반영하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Linked Issues check ✅ Passed 직접 연결된 이슈 #370의 코딩 요구사항을 충족합니다. TimelineQueryService.toRecordTimelineItem은 동일 서재에 기록이 있으면 기존 previewText의 null 또는 공백 여부와 관계없이 최신 기록 미리보기를 사용합니다. 추가된 getTimelinePreview_latestRecord 매개변수 테스트는 목록 …
Out of Scope Changes check ✅ Passed 변경 범위는 #370과 직접 연결됩니다. 서비스의 미리보기 선택 조건 수정과 해당 동작을 검증하는 테스트만 포함됩니다. 목록 및 요약 미리보기의 최신 내용 반영과 본문 부재 시 이미지 개수 표시를 벗어난 변경은 확인되지 않습니다.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@github-actions

Copy link
Copy Markdown
Overall Project 79.59% 🍏
Files changed 100% 🍏

File Coverage
TimelineQueryService.java 97.19% 🍏

@kjhyeon0620
kjhyeon0620 merged commit 56f33f5 into develop-demo Sep 20, 2026
2 checks passed
@kjhyeon0620
kjhyeon0620 deleted the fix/#370-record-timeline-preview branch September 20, 2026 17:19
@kjhyeon0620 kjhyeon0620 mentioned this pull request Sep 21, 2026
4 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] 독서 기록 수정 내용이 독서 히스토리 미리보기에 반영되지 않음

1 participant