[fix/#51] QA 이슈 수정 #52
Conversation
- coerceIn이 (0,-1) 이 되버려서 오류 발생 - null 보내주고, FairySelectionBottomSection 에서 selectedFairy가 null 인 경우 다시 대화하기 만 노출시키도록 개선
Walkthrough채팅 화면의 요정 선택 섹션에서 선택 계산을 안전화하고 버튼/레이아웃 문구를 수정했습니다. 온보딩 7단계 말풍선의 문구를 간단히 변경했습니다. 외부 공개 시그니처 변경은 없습니다. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant U as User
participant CS as ChattingScreen
participant FS as FairySelectionSection
participant BTN as BottomActions
U->>CS: 화면 진입
CS->>FS: 요정 목록 전달
FS->>FS: selectedFairy = list.getOrNull(index)
alt 요정 선택됨
FS-->>BTN: 확인 버튼 활성/문구 업데이트
FS-->>BTN: "다시 대화하기" 표시
else 선택 없음 또는 빈 목록
FS-->>BTN: 확인 버튼 비활성/미표시
note right of BTN: 선택 없음 상태 유지
end
U->>BTN: 확인 탭
alt selectedFairy != null
BTN->>CS: 확정 처리
else
BTN-->>U: 무응답/비활성
end
U->>BTN: 다시 대화하기 탭
BTN->>CS: 재시작 흐름 트리거
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. 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. Comment |
| text = "내 감정은 ${selectedFairy.emotion}이야", | ||
| text = "내 감정은 ${selectedFairy.emotion}인 것 같아", |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
feature/onboarding/src/commonMain/kotlin/com/nexters/emotia/feature/onboarding/OnBoardingScreen.kt (2)
263-267: 문체 일관성 확인 필요 (‘있어요’ → ‘있어’ 변경).7단계만 반말로 변경되었고, 0단계/4단계는 여전히 존댓말이 섞여 있어 톤이 혼재됩니다. 온보딩 내 내레이션/캐릭터 보이스 톤 가이드에 맞춰 전 단계의 문체를 통일할지 확인 부탁드립니다. 의도한 전환이라면 OK, 아니라면 7단계를 원문(“…숨겨져 있어요.”)으로 되돌리거나, 다른 스텝도 반말로 정리하는 방안을 권장합니다.
256-268: 하드코딩된 문구의 리소스화 제안.말풍선 텍스트가 코드에 직접 하드코딩되어 있어 i18n/카피 수정을 반복할 때 변경 범위가 커집니다. 문자열 리소스로 분리하면 번역/카피 수정, A/B 테스트, 접근성(콘텐츠 설명 일관화)에 유리합니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
-
feature/chatting/src/commonMain/kotlin/com/nexters/emotia/feature/chatting/ChattingScreen.kt(2 hunks) -
feature/onboarding/src/commonMain/kotlin/com/nexters/emotia/feature/onboarding/OnBoardingScreen.kt(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: build
- GitHub Check: Firebase App Distribution
🔇 Additional comments (3)
feature/chatting/src/commonMain/kotlin/com/nexters/emotia/feature/chatting/ChattingScreen.kt (3)
432-438: 빈 요정 리스트에 대한 안전한 처리가 잘 적용됨빈 요정 리스트일 때
selectedFairy가null로 설정되도록 변경하여, 이전에 발생했던coerceIn(0, -1)크래시 문제를 효과적으로 방지했습니다.
541-541: 확정 버튼 문구 개선"내 감정은 ${selectedFairy.emotion}이야"에서 "내 감정은 ${selectedFairy.emotion}인 것 같아"로 변경하여 더 부드럽고 자연스러운 어조로 개선되었습니다.
547-560: "다시 대화하기" 옵션 항상 표시로 변경이전에는 조건부로 표시되었던 "다시 대화하기" 옵션이 이제 요정 선택 상태와 관계없이 항상 표시되도록 변경되었고, 적절한 간격(24dp)도 추가되어 UI 일관성이 향상되었습니다.
작업 내용
이건 꼭 봐주세요
Summary by CodeRabbit
버그 수정
스타일