Skip to content

Conversation

@ProtossManse
Copy link
Collaborator

@ProtossManse ProtossManse commented Nov 19, 2025

🔗 관련 이슈

Summary by CodeRabbit

릴리스 노트

  • Bug Fixes
    • 로그인 절차 중 뒤로가기 네비게이션 동작을 개선했습니다.
    • 중간 화면들이 백스택에서 적절하게 제거되도록 수정했습니다.

@coderabbitai
Copy link

coderabbitai bot commented Nov 19, 2025

Walkthrough

로그인 흐름의 백 스택 동작이 수정되었습니다. LoginRegisterUserInfo에서 LoginRegisterElder로 이동할 때, popUpTo(Route.LoginStart)를 추가하여 중간 화면들을 제거하도록 변경되었습니다.

Changes

Cohort / File(s) Change Summary
Navigation Back Stack Management
app/src/main/java/com/konkuk/medicarecall/ui/navigation/NavGraph.kt
Route.LoginRegisterElder 이동 시 popUpTo(Route.LoginStart) 추가로 백 스택 정리 로직 구현

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant NavGraph
    participant BackStack
    
    User->>NavGraph: LoginRegisterUserInfo에서<br/>LoginRegisterElder로 이동
    Note over NavGraph: popUpTo(Route.LoginStart) 실행
    NavGraph->>BackStack: LoginStart까지의<br/>중간 스크린 제거
    BackStack-->>NavGraph: 스택 정리 완료
    NavGraph-->>User: LoginRegisterElder 표시
Loading

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • 단일 파일의 네비게이션 로직 수정
  • 백 스택 동작 변경으로 인한 UI 흐름 검증 필요

Poem

🧭 백 스택이 깔끔해지고,
로그인 길도 순탄해지네,
중간 화면은 사라지고,
더 나은 흐름 완성 ✨


📜 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 2089dc6 and ecad2d0.

📒 Files selected for processing (1)
  • app/src/main/java/com/konkuk/medicarecall/ui/navigation/NavGraph.kt (1 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 34ec176 into develop Nov 20, 2025
2 checks passed
@ProtossManse ProtossManse deleted the fix/navigation-after-register-#195 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] 회원가입 후 뒤로가기 시 navigation 관련 문제

4 participants