Skip to content

캘린더탭 진입 시, 앱 강제 종료 이슈 핫픽스 - #269

Merged
harry7408 merged 1 commit into
developfrom
hot-fix/calendar-error
Feb 3, 2026
Merged

harry7408 merged 1 commit into
developfrom
hot-fix/calendar-error

Conversation

@sunjoolee

@sunjoolee sunjoolee commented Feb 3, 2026 •

Copy link
Copy Markdown
Member

작업 상세 내용

  • 원인: 프리뷰 생성용 더미 데이터 처리 시 에러 발생 (2월 30일 - Invalid Date Error)

Summary by CodeRabbit

릴리즈 노트

  • 개선사항
    • 시간 캡슐 캘린더 컴포넌트의 테스트 데이터를 정리했습니다.

@sunjoolee sunjoolee added the bug Something isn't working label Feb 3, 2026
@coderabbitai

coderabbitai Bot commented Feb 3, 2026 •

Copy link
Copy Markdown
Contributor

Walkthrough

타임캡슐 캘린더 컴포넌트에서 더미 데이터셋을 제거하고, 미리보기에서 빈 리스트를 사용하도록 업데이트했습니다. 실제 데이터 바인딩만 유지되며 하드코딩된 테스트 데이터는 제거되었습니다.

Changes

Cohort / File(s) Summary
TimeCapsule Calendar Preview 정리
feat/time-capsule/src/main/java/com/emotionstorage/time_capsule/ui/component/TimeCapsuleCalendar.kt
DUMMY_TIME_CAPSULE_DATES 데이터셋 제거 및 TimeCapsuleCalendarPreview에서 빈 리스트로 대체

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3분

Suggested reviewers

  • harry7408

안녕하세요! 이번 변경사항은 정말 깔끔하네요. 타임캡슐 캘린더의 더미 데이터를 제거하고 프리뷰를 간결하게 정리한 거군요.

변경사항이 간단 명료하고, 실제 기능 동작에는 영향이 없으니 빠르게 검토될 수 있을 것 같습니다. 혹시 이제 실제 데이터 바인딩에만 집중하게 되는 건가요? 그렇다면 정말 좋은 리팩토링 작업입니다! 😊

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 변경 내용의 핵심을 정확하게 반영하고 있습니다. 캘린더 진입 시 강제 종료 이슈와 그 해결책(핫픽스)을 명확하게 전달합니다.
Description check ✅ Passed PR 설명이 기본 구조를 갖추고 있으나, 필수 섹션인 'Related Issue'가 누락되어 있습니다. 작업 상세 내용은 원인과 조치를 간단히 설명하고 있습니다.

✏️ 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 hot-fix/calendar-error

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.

@harry7408 harry7408 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@harry7408
harry7408 merged commit 2b40efe into develop Feb 3, 2026
3 checks passed
@sunjoolee
sunjoolee deleted the hot-fix/calendar-error branch February 3, 2026 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants