Skip to content

Conversation

@ProtossManse
Copy link
Collaborator

@ProtossManse ProtossManse commented Nov 19, 2025

🔗 관련 이슈

📙 작업 설명

  • 이제 popUpTo(0) 를 사용해서 홈으로 이동할 때 온보딩 backStack을 전부 삭제합니다.

Summary by CodeRabbit

  • 버그 수정
    • 로그인 후 홈 화면으로 이동할 때 네비게이션 흐름을 표준화하여 모든 로그인 경로에서 일관된 동작을 제공합니다.
    • 이전의 복잡한 백스택 복원 동작을 제거하고 단일 루트로 초기화 후 홈으로 이동하도록 개선했습니다.
    • 뒤로 가기 버튼 동작이 더 예측 가능하고 안정적으로 동작하도록 수정했습니다.
    • 전반적인 화면 전환 신뢰성을 향상시켜 로그인 경험이 일관되게 유지됩니다.

@coderabbitai
Copy link

coderabbitai bot commented Nov 19, 2025

Walkthrough

로그인 완료 시 여러 경로에서 사용되던 메인 이동 로직을 navigateToMainAfterLogin()로 통합하고, 호출 시 루트까지 팝(pop, inclusive=true)한 뒤 MainTabRoute.Home으로 이동하도록 동작을 변경했습니다. 백스택 복원 관련 옵션(restoreState, launchSingleTop)은 제거되었습니다.

Changes

코호트 / 파일(s) 변경 요약
네비게이션 로직 중앙화
app/src/main/java/com/konkuk/medicarecall/ui/navigation/NavGraph.kt
navigateToMainAfterLogin()의 본문을 수정하여 루트(0)까지 popBackStack(..., inclusive = true) 호출 후 MainTabRoute.Home으로 네비게이션하도록 표준화. Splash, LoginStart, LoginVerification의 직적 호출들을 이 헬퍼로 통합. restoreStatelaunchSingleTop 제거.

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant AuthScreen as Splash / LoginStart / LoginVerification
    participant NavGraph
    participant NavController
    participant Home as MainTabRoute.Home

    User->>AuthScreen: 로그인 완료
    AuthScreen->>NavGraph: navigateToMainAfterLogin()
    NavGraph->>NavController: popBackStack(root, inclusive=true)
    NavController-->>NavGraph: 백스택 정리 완료
    NavGraph->>NavController: navigate(MainTabRoute.Home)
    NavController->>Home: Home 화면 표시
    Home->>User: 렌더링
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8분

  • 단일 파일에서 일관된 네비게이션 로직 변경이므로 리뷰 부담이 낮음.
  • 주의 포인트:
    • 로그인 흐름(Splash, LoginStart, LoginVerification)에서 동일 동작이 적용되는지 확인
    • 루트까지 팝하는 동작이 의도치 않은 화면 제거를 일으키지 않는지 검증
    • 상태 복원 관련 제거로 인한 UX 변화 여부 확인

루트로 한걸음, 길을 정리하니
흩어진 호출 하나로 모여드네
Back 스택 던져두고 가볍게 Home으로 —
코드도 마음도 한결 편안하네 🏠✨


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between 553df27 and 10ce85d.

📒 Files selected for processing (1)
  • app/src/main/java/com/konkuk/medicarecall/ui/navigation/NavGraph.kt (4 hunks)

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

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

Copy link
Contributor

@ikseong00 ikseong00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Collaborator

@alswlekk alswlekk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ProtossManse ProtossManse merged commit 4e1eb0d into develop Nov 20, 2025
2 checks passed
@ProtossManse ProtossManse deleted the fix/verification-to-home-nav-#190 branch November 20, 2025 06:14
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.

[Fix] 휴대폰 인증 -> 홈 이동시 뒤로가기 네비게이션 버그

4 participants