Skip to content

fix: 홈 이탈 시 대화방 자동 이동 이벤트 무시 - #64

Merged
soyeonLee126 merged 5 commits into
developfrom
fix/home-qa
Aug 21, 2026
Merged

soyeonLee126 merged 5 commits into
developfrom
fix/home-qa

Conversation

@seunghee17

@seunghee17 seunghee17 commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

작업 개요

홈 화면에서 닉네임을 변경한 뒤 돌아왔을 때 최신 닉네임이 반영되지 않는 문제와, 홈 화면에서 메시지를 전송한 뒤 다른 화면으로 이탈했음에도 늦게 완료된 전송 결과로 대화방에 자동 이동하는 문제를 수정합니다.

특히 Navigation3의 top-level entry는 화면이 보이지 않는 동안에도 유지될 수 있어 HomeViewModel과 화면의 collector가 계속 살아 있을 수 있습니다. 이 상태에서 기존 Orbit side effect로 대화방 이동을 전달하면 사용자의 현재 위치와 무관한 stale navigation이 발생할 수 있습니다.

작업 유형

  • feat: 새 기능
  • fix: 버그 수정
  • refactor: 리팩토링 (동작 변경 없음)
  • build: 빌드/의존성 설정
  • chore: 기타 (문서, 설정 등)
  • test: 테스트 추가/수정

변경 사항

  • HomeScreen 진입 시 loadUserInfo()를 호출해 설정 화면에서 변경한 닉네임을 다시 조회
  • HomeViewModel의 닉네임 초기 로딩을 명시적인 loadUserInfo() 함수로 분리
  • 닉네임 재조회 실패 시 기존에 표시 중인 닉네임을 유지
  • 홈 화면의 대화방 이동 이벤트를 Orbit side effect에서 MutableSharedFlow(replay = 0)로 분리
  • HomeScreen이 활성 상태일 때만 대화방 이동 이벤트를 collect하도록 변경
  • HomeScreen 이탈 중 발생한 이벤트가 재진입 시 재생되지 않도록 처리
  • MainScreen에서 HomeScreen에 현재 top-level 활성 상태를 전달
  • 닉네임 갱신, 재조회 실패, 대화방 이동 이벤트, 홈 이탈 시 stale navigation 관련 ViewModel 테스트 보완

관련 이슈

  • 해당 없음

관련 작업 (Notion)

  • 해당 없음

스크린샷 / 동작 화면

  • UI 변경 없음

체크리스트

  • base 브랜치가 develop으로 설정되어 있다
  • 로컬 빌드 및 실행을 확인했다
  • 커밋 메시지가 컨벤션(feat:, fix: 등)을 따른다
  • 불필요한 로그/주석/디버그 코드를 제거했다
  • 리뷰어가 이해할 수 있도록 설명을 작성했다

리뷰 요청 사항

  • Top-level Navigation entry가 유지되는 상황에서 Home 이탈 후 늦게 완료된 메시지 전송이 대화방 자동 이동을 발생시키지 않는지 확인 부탁드립니다.
  • SharedFlow(replay = 0)를 이용한 화면 활성 상태 기반 일회성 navigation event 처리가 현재 Navigation3 구조에 적절한지 확인 부탁드립니다.
  • 닉네임 변경 후 HomeScreen 재진입 시 사용자 정보 재조회 및 실패 시 기존 상태 유지가 적절한지 확인 부탁드립니다.

검증

  • ./gradlew test
  • ./gradlew detekt lint

@seunghee17
seunghee17 marked this pull request as ready for review August 21, 2026 12:37
- 홈 메시지 전송시 대화방 이동 이벤트 검증
- 홈 이탈 후 대화방 이동 이벤트 재생 방지 검증
@soyeonLee126
soyeonLee126 merged commit 2dca02d into develop Aug 21, 2026
6 checks passed
@soyeonLee126
soyeonLee126 deleted the fix/home-qa branch September 1, 2026 07:45
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.

2 participants