Skip to content

feat: 컬렉션 후속 작업 - #963

Merged
devfeijoa merged 1 commit into
developfrom
feat/961
Sep 10, 2026
Merged

feat: 컬렉션 후속 작업#963
devfeijoa merged 1 commit into
developfrom
feat/961

Conversation

@devfeijoa

@devfeijoa devfeijoa commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • full build 시 에러 난 부분 수정

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

Summary by CodeRabbit

  • 버그 수정
    • 공유 데이터가 오래된 경우 스낵바 오류 메시지가 보다 안정적으로 표시되도록 개선했습니다.
    • 오류 메시지 처리 방식을 정리해 화면 내 안내가 일관되게 표시됩니다.

- `CollectionDetailScreen.kt`:
    - `LocalContext` 제거 및 `stringResource`를 사용하여 스낵바 메시지를 관리하도록 변경
    - `LaunchedEffect` 내 에러 발생 시 스낵바 노출 조건을 명확하게 수정
    - 공유 데이터 만료 메시지(`collection_share_data_outdated`)를 변수로 추출하여 재사용성 향상
@coderabbitai

coderabbitai Bot commented Sep 10, 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: 4259c7e5-7b28-493b-b226-da04b665d32e

📥 Commits

Reviewing files that changed from the base of the PR and between fa9003f and 6d7efdb.

📒 Files selected for processing (1)
  • feature/collection/src/main/java/com/into/websoso/feature/collection/CollectionDetailScreen.kt

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


Walkthrough

컬렉션 상세 화면에서 LocalContext 사용을 제거했습니다. 문자열 리소스를 미리 생성하고, 오류 및 공유 콘텐츠 생성 실패 시 해당 문자열을 스낵바에 표시하도록 변경했습니다.

Changes

컬렉션 상세 메시지 처리

Layer / File(s) Summary
스낵바 메시지 사전 생성 및 오류 처리
feature/collection/.../CollectionDetailScreen.kt
stringResource로 메시지를 미리 생성합니다. LaunchedEffect에서 조건을 확인한 뒤 오류를 소비하고 스낵바를 표시합니다. 공유 콘텐츠 생성 실패에도 사전 생성한 메시지를 사용합니다.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to 6d7ef

The collection detail screen now resolves snackbar strings through Compose resources while retaining guarded error handling; no user-facing regression or merge-blocking risk is identified.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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 1 functions across 1 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
Linked Issues check ❓ Inconclusive 변경 내용은 컬렉션 기능의 후속 작업이라는 이슈 범위와 일치합니다. 그러나 이슈 #961에 구체적인 요구사항이나 완료 조건이 없어 요구사항 충족 여부를 완전히 확인할 수 없습니다. 이슈 #961에 구체적인 요구사항 또는 완료 조건을 추가하거나, PR 설명에 이번 변경의 목적과 검증 결과를 명시하세요.
✅ Passed checks (3 passed)
Check name Status Explanation
Out of Scope Changes check ✅ Passed 변경은 CollectionDetailScreen의 리소스 참조와 오류 메시지 처리 개선에 한정됩니다. 컬렉션 기능 후속 작업 범위를 벗어난 변경은 확인되지 않습니다.
Title check ✅ Passed 제목은 컬렉션 기능의 후속 작업이라는 PR의 주요 목적과 관련됩니다. 변경 내용을 구체적으로 설명하지는 않지만, 실제 변경 범위와 일치합니다.
Description check ✅ Passed 설명은 이슈 번호와 작업 내용 섹션을 포함하며, 전체 빌드 오류 수정이라는 변경 목적을 설명합니다. 스크린샷과 리뷰어 안내 내용은 비어 있지만, 핵심 정보는 제공되어 설명은 대체로 완전합니다.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/961

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

토끼가 문자열을 미리 담고
스낵바 길을 가볍게 닦네
오류도 조용히 소비되고
공유 실패 알림도 반짝
Context는 둥지를 떠나
컬렉션 화면이 포근해졌네

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

@devfeijoa devfeijoa changed the title feat/961 feat: 컬렉션 후속 작업 Sep 10, 2026
@devfeijoa
devfeijoa merged commit bf148e0 into develop Sep 10, 2026
3 checks passed
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.

feat : 컬렉션 후속 작업

2 participants