Skip to content

refactor(timeletter): 단순 back 위임 5개를 popBack 하나로 합친다 (#1301) - #2306

Closed
OnlyReadOneBook wants to merge 1 commit into
developfrom
refactor/1301-timeletter-pop-back
Closed

OnlyReadOneBook wants to merge 1 commit into
developfrom
refactor/1301-timeletter-pop-back

Conversation

@OnlyReadOneBook

@OnlyReadOneBook OnlyReadOneBook commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

핵심 검토 파일: TimeLetterNavActions.kt, TimeLetterNavGraph.kt (프로덕션)

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

🧪 CI Test Plan

{
  "androidTest": {
    "mode": "selected",
    "reason": "navigation/ 경로 변경. 타임레터 작성·수신인 선택·상세의 뒤로가기 경로를 지나는 계측 테스트를 고른다.",
    "tests": [
      {"path": "app/src/androidTest/java/com/afternote/afternote_fe/TimeLetterMindRecordCompletionAndroidTest.kt", "selector": "com.afternote.afternote_fe.TimeLetterMindRecordCompletionAndroidTest#timeLetterRecipientSelector_roundTripPreservesTitleTextAndExactReceiverId", "device": "api34"},
      {"path": "app/src/androidTest/java/com/afternote/afternote_fe/TimeLetterMindRecordCompletionAndroidTest.kt", "selector": "com.afternote.afternote_fe.TimeLetterMindRecordCompletionAndroidTest#timeLetterWrite_uiValidationAndRapidRegister_preserveInputAndCreateOnce", "device": "api34"},
      {"path": "app/src/androidTest/java/com/afternote/afternote_fe/TimeLetterMindRecordCompletionAndroidTest.kt", "selector": "com.afternote.afternote_fe.TimeLetterMindRecordCompletionAndroidTest#timeLetterSenderDetail_routeIdLoadingFailureRetryAndSuccess_areConnected", "device": "api34"}
    ]
  }
}

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

화면 변경 없음.

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

로컬 검증: 타임레터 presentation 컴파일, app 단위 테스트 컴파일, ktlintCheck, app 네비게이션 단위 테스트 통과.

@github-actions github-actions Bot added android-test PR head에서 Android Managed Device 계측 테스트 실행 sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 labels Oct 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

열린 PR 중 이 PR 과 같은 파일을 서로 다르게 고친 것이 있다. 어느 쪽이 먼저 머지되든 나중 쪽은 base 를 병합해야 통과한다 (#1750).

머지 자체는 GitHub 이 막는다 — 한쪽이 들어가면 다른 쪽은 CONFLICTING 이 되어 merge queue 에 못 들어간다.
이 라벨은 그 전에 순서를 정하라는 신호다. 먼저 갈 쪽을 정하고, 나중 쪽은 그 머지 뒤에 base 를 병합한다.
상대 PR 에도 같은 안내가 달린다.

이 코멘트는 리컨사일러가 매 실행 현재 상태로 고쳐 쓴다. 해소되면 sibling-conflict 라벨은 자동으로 떨어진다.

@OnlyReadOneBook

Copy link
Copy Markdown
Collaborator Author

타임레터 모듈 작업이라 #1301 을 @koongmai 에게 넘기고 이 PR 은 닫습니다. 브랜치 refactor/1301-timeletter-pop-back 은 참고용으로 남겨 둡니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

android-test PR head에서 Android Managed Device 계측 테스트 실행 sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

1 participant