Skip to content

[feat] 테스트 메뉴에서 레벨업 축하 다이얼로그 미리보기 - #165

Merged
UiHyeon-Kim merged 1 commit into
developfrom
feat/test-menu-level-up-preview
Sep 19, 2026
Merged

UiHyeon-Kim merged 1 commit into
developfrom
feat/test-menu-level-up-preview

Conversation

@UiHyeon-Kim

Copy link
Copy Markdown
Member

작업 요약

레벨업 축하 다이얼로그는 실제로 주행 거리를 쌓아 레벨이 올라야만 떠서, 내부 테스터나 기획·디자인이 확인할 방법이 없었습니다. 테스트 메뉴에서 레벨을 골라 바로 띄울 수 있게 했습니다.

주요 변경사항

  • 마이페이지 > 테스트 > 레벨업 축하 섹션 추가: Seed / Rookie / Owner / Explorer / Navigator
  • 누르면 테스트 메뉴가 닫히고 홈에서 쓰는 것과 같은 LevelUpDialog 가 그 레벨로 뜹니다. 확인이나 바깥 터치로 닫힙니다
  • 화면에 띄우기만 하고 서버·로컬 상태는 건드리지 않아, 눌러도 실패하거나 계정이 바뀌는 일이 없습니다

리뷰 포인트

  • 홈 화면 상태에 레벨업을 주입하는 대신, 같은 컴포넌트를 마이페이지 위에 띄웠습니다. 다이얼로그는 별도 창으로 떠서 보이는 모습이 같고, 홈 ViewModel에 테스트용 경로를 만들지 않아도 됩니다
  • 레벨 라벨은 마이페이지 프로필 카드와 같은 표기(첫 글자만 대문자)를 씁니다

검증

  • assembleDebug test verifyRoborazziDebug lint --rerun-tasks 전체 통과, check-conventions.sh BLOCK 통과(WARN 2건, 변경 없음)
  • 에뮬레이터(API 36.1): 테스트 메뉴에 "레벨업 축하" 섹션 노출, Rookie·Navigator 다이얼로그가 해당 캐릭터와 라벨로 표시되고 확인 버튼으로 닫히는 것 확인

🤖 Generated with Claude Code

레벨업 축하는 주행 거리를 쌓아야만 떠서 디자인을 확인할 방법이 없었다.
홈과 같은 LevelUpDialog를 레벨 5종 중 골라 띄운다. 서버 상태는 건드리지 않는다.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 19, 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: Repository: Central-MakeUs/Rodi-Android/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 19d8e4f9-0027-4f11-a8a8-3b11f4415d25

📥 Commits

Reviewing files that changed from the base of the PR and between 9151303 and 686c2b1.

📒 Files selected for processing (1)
  • feature/mypage/src/main/java/com/dororong/rodi/feature/mypage/MyPageScreen.kt

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


📝 Walkthrough

Walkthrough

MyPageScreenOnboardingLevel.entries 기반의 레벨업 테스트 메뉴를 추가했습니다. 메뉴에서 레벨을 선택하면 메뉴를 닫고 previewLevelUp 상태를 설정합니다. 상태가 설정되면 해당 레벨의 LevelUpDialog를 표시합니다. 확인 또는 닫기 요청이 발생하면 상태를 초기화합니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 686c2

The preview uses consistent level labels and correctly closes through the supported actions, so no actionable merge risk remains.

🚥 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%. 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.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 테스트 메뉴에서 레벨업 축하 다이얼로그를 미리 보는 핵심 변경을 정확하고 간결하게 설명합니다.
Description check ✅ Passed PR 설명은 레벨 선택, 다이얼로그 표시와 종료 동작, 상태 변경 없음, 검증 결과를 변경 사항과 일치하게 설명합니다.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@UiHyeon-Kim
UiHyeon-Kim merged commit 5a3baa0 into develop Sep 19, 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.

1 participant