fix: 홈 이탈 시 대화방 자동 이동 이벤트 무시 - #64
Merged
Merged
Conversation
seunghee17
marked this pull request as ready for review
August 21, 2026 12:37
- 홈 메시지 전송시 대화방 이동 이벤트 검증 - 홈 이탈 후 대화방 이동 이벤트 재생 방지 검증
soyeonLee126
approved these changes
Aug 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
작업 개요
홈 화면에서 닉네임을 변경한 뒤 돌아왔을 때 최신 닉네임이 반영되지 않는 문제와, 홈 화면에서 메시지를 전송한 뒤 다른 화면으로 이탈했음에도 늦게 완료된 전송 결과로 대화방에 자동 이동하는 문제를 수정합니다.
특히 Navigation3의 top-level entry는 화면이 보이지 않는 동안에도 유지될 수 있어 HomeViewModel과 화면의 collector가 계속 살아 있을 수 있습니다. 이 상태에서 기존 Orbit side effect로 대화방 이동을 전달하면 사용자의 현재 위치와 무관한 stale navigation이 발생할 수 있습니다.
작업 유형
변경 사항
loadUserInfo()를 호출해 설정 화면에서 변경한 닉네임을 다시 조회loadUserInfo()함수로 분리MutableSharedFlow(replay = 0)로 분리MainScreen에서 HomeScreen에 현재 top-level 활성 상태를 전달관련 이슈
관련 작업 (Notion)
스크린샷 / 동작 화면
체크리스트
develop으로 설정되어 있다feat:,fix:등)을 따른다리뷰 요청 사항
SharedFlow(replay = 0)를 이용한 화면 활성 상태 기반 일회성 navigation event 처리가 현재 Navigation3 구조에 적절한지 확인 부탁드립니다.검증
./gradlew test./gradlew detekt lint