Skip to content

TimeLetter 단순 back 위임 5개를 공통 명령으로 통합 #1301

Open

Description

@OnlyReadOneBook

#924의 단순 back 위임 정리 중 TimeLetter 담당 범위를 추적한다.

Setting의 단순 back 15개와 루트 adapter 정리는 #1695 / PR #1962의 로컬 Navigation 3 이관이 대체했다. Setting은 SettingLocalNavActions.popBack()으로 통합됐고 로그아웃·탈퇴는 별도 루트 경계로 남는다. 이 이슈에 같은 Setting 완료 조건을 중복 추적하지 않는다. MindRecord는 #1311에서 추적한다.

남은 범위

  • TimeLetterNavActions의 단순 back 5개를 공통 popBack() 명령으로 통합한다.
  • TimeLetter Nav3 이관 #1696의 최종 entry/actions 계약 위에서 저장·등록·수신자·필터 완료 콜백을 연결한다. 새 Nav2 adapter를 만든 뒤 #1696에서 다시 쓰지 않는다.
  • 현재 Nav2의 화면 이벤트 이름·저장/등록 이벤트 소비 후 pop 순서·수신자/필터 상태 반영 후 pop 순서를 보존한다.
  • 더 이상 쓰지 않는 TimeLetter 구현·호출부·문서를 제거한다.
  • 관련 컴파일·ktlint·Konsist 및 back-stack Android 검증을 통과한다.

선행과 완료

  • 실제 blocked_by는 기존 TimeLetter 구조 선행 #1696만 유지한다. Setting refactor(setting): 설정 흐름을 Navigation 3 로컬 스택으로 이관 #1695 의존은 범위 제거와 함께 해제한다.
  • #1696이 TimeLetter의 공통 back 계약까지 구현하면 그 결과로 이 이슈를 대체·종료하고 중복 PR을 만들지 않는다. 남은 개선이 있을 때만 별도 PR로 완료한다.
  • 이 이슈가 끝날 때 TimeLetter 단순 back 구현은 공통 명령 하나이며 저장 성공·수신자 선택·필터 복귀의 기존 결과가 유지된다.

범위 밖

관련: #924, #1311, #1695, #1696. 기존 혼합 PR #1308의 폐기 구현은 재사용하지 않는다.

Activity

  1. changed the title [-]refactor(app): NavActions 단순 back 위임 28개를 기능별 공통 명령으로 통합[/-] [+]refactor(navigation): Setting·TimeLetter 단순 back 위임 20개를 공통 명령으로 통합[/+] on Aug 27, 2026
  2. changed the title [-]refactor(navigation): Setting·TimeLetter 단순 back 위임 20개를 공통 명령으로 통합[/-] [+]Setting·TimeLetter 단순 back 위임 20개를 공통 명령으로 통합[/+] on Aug 28, 2026
  3. added
    refactor동작 변경 없이 코드 구조·책임·경계를 개선하는 작업
    and removed
    enhancementNew feature or request
    on Aug 28, 2026
  4. OnlyReadOneBook commented on Sep 2, 2026

    @OnlyReadOneBook
    CollaboratorAuthor

    blocked_by #1695 · #1696 을 걸었다.

    이 이슈가 바꾸는 계층을 #1695·#1696 이 통째로 재설계한다 — 범위가 겹치는 게 아니라 포함된다.

    이 이슈 #1695 · #1696
    SettingNavActions back 15개 → popBack() SettingRoute 를 Nav3 key 계약으로, SettingNavGraph 등록을 entryProvider 로 대체
    TimeLetterNavActions back 5개 → popBack() TimeLetterNavGraph 의 previousBackStackEntry VM owner 계약 재설계
    두 NavGraph 콜백 32곳 매핑 두 그래프의 화면 등록 전체 이관
    AppNavigationActions.kt adapter 변경 Route.Setting·Route.TimeLetter 만 루트 Nav2 경계로 남김
    Setting·TimeLetter back-stack androidTest 추가 로컬 NavBackStack 기준으로 재작성

    Nav3 로컬 스택에서 back 은 backStack.removeLastOrNull() 이라 위임 계층 자체가 사라진다. 20개를 1개로 줄여도 그 1개를 #1695·#1696 이 다시 쓴다 — 선행 통합은 이중 작업이다.

    #1695·#1696 이 닫히면 이 이슈는 「남은 위임이 있는가」로 다시 판정한다. 없으면 닫는다.

  5. changed the title [-]Setting·TimeLetter 단순 back 위임 20개를 공통 명령으로 통합[/-] [+]TimeLetter 단순 back 위임 5개를 공통 명령으로 통합[/+] on Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

area:timeletter타임레터 기능 영역refactor동작 변경 없이 코드 구조·책임·경계를 개선하는 작업

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions