[doyeonk429]

🔗 관련 이슈

• Close [BOOK-509/feat] v1.4.0 온보딩 화면 수정사항 적용 #273

📘 작업 유형

• ✨ Feature (기능 추가)
• 🐞 Bugfix (버그 수정)
• 🔧 Refactor (코드 리팩토링)
• ⚙️ Chore (환경 설정)
• 📝 Docs (문서 작성 및 수정)
• ✅ Test (기능 테스트)
• 🎨 style (코드 스타일 수정)

📙 작업 내역

• 이미지 에셋 교체 및 문구 수정
• 피그마 디자인 가이드 대로 이미지 사이즈 비율 적용 및 간격 수정

🧪 테스트 내역

• 브라우저/기기에서 동작 확인
• 엣지 케이스 테스트 완료
• 기존 기능 영향 없음

🎨 스크린샷 또는 시연 영상 (선택)

기능	미리보기	미리보기	미리보기
온보딩

✅ PR 체크리스트

• 커밋 메시지가 명확합니다
• PR 제목이 컨벤션에 맞습니다
• 관련 이슈 번호를 작성했습니다
• 기능이 정상적으로 작동합니다
• 불필요한 코드를 제거했습니다

Summary by CodeRabbit

릴리스 노트

• UI 및 콘텐츠 개선
  • 온보딩 이미지의 높이와 레이아웃이 조정되어 더 나은 시각적 구성을 제공합니다.
  • 온보딩 페이지의 제목, 설명, 강조 텍스트 등 모든 사용자 대면 콘텐츠가 새로 업데이트되었습니다.
  • 인터페이스의 여백과 간격 설정이 최적화되었습니다.

[coderabbitai]

Walkthrough

온보딩 화면을 v1.4.0 버전으로 업데이트합니다. 이미지 에셋 파일명을 "graphic"에서 "onboarding2"로 변경하고, 온보딩 뷰의 텍스트 문구를 수정하며, 이미지 높이 제약 조건(너비 대비 274/375 비율)과 간격 상수를 조정합니다.

Changes

Cohort / File(s)	Summary
이미지 에셋 src/Projects/BKDesign/Resources/Assets.xcassets/graphics/onboarding2.imageset/Contents.json	모든 해상도(1x, 2x, 3x)에 대해 파일명을 "graphic"에서 "onboarding2"로 변경합니다.
온보딩 뷰 UI 및 레이아웃 src/Projects/BKPresentation/Sources/OnboardingFlow/View/OnboardingView.swift	이미지 뷰 높이 제약 조건 추가(너비의 274/375), 레이블 스택 상단 간격 조정(spacing7 → spacing8), 온보딩 페이지의 제목, 설명, 강조 단어 등 텍스트 문구 업데이트입니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 2

❌ Failed checks (2 warnings)

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.
Merge Conflict Detection	⚠️ Warning	⚠️ Unable to check for merge conflicts: Invalid branch name format

✅ 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	PR의 모든 변경사항이 이슈 #273의 요구사항을 충족합니다: 이미지 에셋 적용 및 텍스트 문구 수정이 완료되었습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 온보딩 페이지 업데이트와 관련되어 있으며, 이슈 #273의 범위 내에서 진행되었습니다.

_{✏️ 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 BOOK-509-feature/#273

⚔️ Resolve merge conflicts (beta)

• Auto-commit resolved conflicts to branch BOOK-509-feature/#273
• Create stacked PR with resolved conflicts
• Post resolved changes as copyable diffs in a comment

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

src/Projects/BKPresentation/Sources/OnboardingFlow/View/OnboardingView.swift (1)

114-118: 매직 넘버를 상수로 추출하는 것을 고려해 보세요.

274.0 / 375.0 비율이 피그마 디자인 가이드에서 온 값이라면, LayoutConstants에 명명된 상수로 추출하면 의도가 더 명확해지고 나중에 변경 시 관리가 편해집니다.

♻️ 제안

 // LayoutConstants enum 내부에 추가
 enum LayoutConstants {
+    static let imageAspectRatio: CGFloat = 274.0 / 375.0
     static let labelStackSpacing = BKSpacing.spacing3
     static let labelStackTopOffset = BKSpacing.spacing8
     static let labelStackHorizontalInset = BKInset.inset5
 }

 imageView.snp.makeConstraints {
     $0.top.equalToSuperview()
     $0.horizontalEdges.equalToSuperview()
-    $0.height.equalTo(imageView.snp.width).multipliedBy(274.0 / 375.0)
+    $0.height.equalTo(imageView.snp.width).multipliedBy(LayoutConstants.imageAspectRatio)
 }

---

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.}