[Feat] 상세 화면 셀프 QA

[giovannijunseokim]

📮 관련 이슈

• closed [Feat] 컬렉션 상세 화면 셀프 QA #162

📌 작업 내용

• feat: 작품 제목은 글자 수 기준이 아닌, 텍스트 박스 width 기준으로 최대 2줄까지 표시한다
• feat: createdAt 표시 형식 변경
• feat: R.drawable.img_collection_bg2 이미지 교체

Summary by CodeRabbit

릴리스 노트

• 새로운 기능

  • 컬렉션 생성일 표시 개선으로 더 읽기 쉬운 날짜 형식 제공
  • 북마크 상태 정보 추가

• 버그 수정

  • 컬렉션 제목이 긴 경우 텍스트 줄임(…) 처리 개선

• 리팩토링

  • 내부 데이터 구조 업데이트 및 정렬

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

데이터 모델 CollectionDetailModel 내 createdAt 타입을 String에서 LocalDate로 변경하고, isBookmarked, title 필드를 추가하며, userId를 private으로 변경했습니다. UI 화면에서 ContentModel을 ContentModelNew로 마이그레이션하면서 필드명을 재구성하고 날짜 서식 지정 로직을 추가했습니다.

Changes

Cohort / File(s)	변경 요약
도메인 모델 업데이트 app/src/main/java/com/flint/domain/model/collection/CollectionDetailModel.kt	createdAt: String → createdAt: LocalDate로 타입 변경; LocalDate.parse() 매핑 로직 추가; isBookmarked: Boolean, title: String 필드 추가; userId를 private 생성자 파라미터로 변경; 불필요한 import 제거 및 LocalDate import 추가
UI 화면 업데이트 app/src/main/java/com/flint/presentation/collectiondetail/CollectionDetailScreen.kt	ContentModel → ContentModelNew 마이그레이션; 날짜 포매팅 로직 추가 (DateTimeFormatter 사용); 헤더 제목 텍스트 오버플로우 처리 (2줄 제한, 말줄임표); 필드명 변경 (contentId → id, posterImage/ottSimpleList → imageUrl, description → reason); bookmarkCount 필드 추가; 미리보기 데이터 업데이트

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• [Feat] CollectionDetailScreen 추가 구현 #153 — CollectionDetailModel의 userId 가시성 변경 및 CollectionDetailScreen의 헤더/북마크 관련 UI 연결 부분에서 겹치는 API 변경사항이 있습니다.
• [Feat] ExploreScreen(탐색 화면) #120 — CollectionDetailScreen과 도메인 콘텐츠 모델 마이그레이션(ContentModel → ContentModelNew) 및 createdAt/날짜 처리 부분에서 동일한 UI와 모델 변경사항이 있습니다.
• [Feat] CollectionDetailScreen 구현 #139 — CollectionDetailModel의 생성자 매핑 로직 및 DTO-to-model 변환 부분에서 직접적으로 관련된 변경사항이 있습니다.

Suggested labels

📱 UI

Suggested reviewers

• nahy-512

Poem

🐰 모델은 새로운 옷을 입고,
날짜는 LocalDate로 변신했네요!
ContentModelNew, 필드들도 단장하고,
UI는 더 예뻐졌어요.
당신의 컬렉션, 이제 완벽해! ✨

🚥 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 제목 '[Feat] 상세 화면 셀프 QA'는 컬렉션 상세 화면의 UI/UX 개선 작업을 수행하는 PR의 주요 목적을 명확하게 반영하고 있습니다.
Description check	✅ Passed	PR 설명이 관련 이슈(#162)를 명시하고, 작품 제목 표시 방식 변경, createdAt 형식 변경, 이미지 교체 등 세 가지 주요 작업 내용을 커밋 링크와 함께 제시하여 템플릿을 충분히 준수합니다.
Linked Issues check	✅ Passed	PR은 issue #162 컬렉션 상세 화면 셀프 QA 요구사항을 충족합니다. 화면 설계서 기준 점검을 통해 제목 표시 방식, 날짜 형식, 배경 이미지 등을 개선하였습니다.
Out of Scope Changes check	✅ Passed	코드 변경사항은 issue #162의 셀프 QA 범위 내에서 이루어졌습니다. CollectionDetailModel과 CollectionDetailScreen의 변경은 모두 UI 표시 형식 개선과 관련된 것입니다.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In
`@app/src/main/java/com/flint/domain/model/collection/CollectionDetailModel.kt`:
- Line 28: The LocalDate.parse(collectionDetail.createdAt) call can throw
DateTimeParseException because it assumes ISO-8601; wrap the parse in
runCatching like the UserRoleType parsing and use a DateTimeFormatter matching
the server format (or a lenient/secondary formatter) so failures are handled
safely and a sensible fallback (null or default date) is returned; update the
createdAt assignment in CollectionDetailModel to use runCatching {
LocalDate.parse(collectionDetail.createdAt, yourFormatter) }.getOrElse { /*
fallback */ } and ensure the formatter constant is clearly named and reused.

🧹 Nitpick comments (1)

app/src/main/java/com/flint/presentation/collectiondetail/CollectionDetailScreen.kt (1)

114-116: DateTimeFormatter 인스턴스 재사용 고려

DateTimeFormatter.ofPattern()은 매 호출 시 새 인스턴스를 생성합니다. remember를 사용하거나 상수로 정의하면 불필요한 객체 생성을 줄일 수 있습니다.

♻️ 제안하는 개선 방안

-private const val DATE_FORMAT_TO_SHOW = "yyyy. MM. dd."
+private val DATE_FORMATTER_TO_SHOW: DateTimeFormatter = DateTimeFormatter.ofPattern("yyyy. MM. dd.")

 // CollectionDetailRoute 내부에서:
-                createdAt = collectionDetail.createdAt.format(
-                    DateTimeFormatter.ofPattern(DATE_FORMAT_TO_SHOW)
-                ),
+                createdAt = collectionDetail.createdAt.format(DATE_FORMATTER_TO_SHOW),

[nahy-512]

고생하셨습니다