Repository navigation
refactor(receiver): 수신자 홈 내려받기 실패 Toast 를 Snackbar 로 전환 (#1391) - #1412
Merged
Merged
Conversation
This was referenced Aug 28, 2026
OnlyReadOneBook
force-pushed
the
refactor/1391-receiver-toast-snackbar
branch
from
August 28, 2026 23:43
3459f10 to
0f44e03
Compare
OnlyReadOneBook
changed the base branch from
develop
to
refactor/1424-receiver-summary-dead
August 28, 2026 23:45
Sadturtleman
approved these changes
Aug 29, 2026
Base automatically changed from
refactor/1424-receiver-summary-dead
to
develop
August 29, 2026 02:12
수신자 홈(ReceiverHomeScreen)·요약(ReceiverAfterNoteEntry)의 내려받기 실패 안내를 android.widget.Toast 에서 Scaffold SnackbarHost 로 옮긴다. 서버 작업 실패의 정본은 #446 공통 에러 팝업이라 컴포넌트가 나오면 재정렬되는 잠정 처분이다(#713 전례, #664 컨벤션). 문구는 기존 상태가 나르는 string resource (receiver_home_download_all_* / receiver_download_all_*)를 그대로 쓴다. Toast 는 Compose semantics 에 잡히지 않아 단언 불가였던 표출을 Robolectric Compose 테스트 3건으로 가드한다. Refs #1391 (건 1·2 — 건 3 은 #1395 머지 후 후속)
OnlyReadOneBook
force-pushed
the
refactor/1391-receiver-toast-snackbar
branch
from
August 29, 2026 02:12
0f44e03 to
4c5b40c
Compare
This was referenced Aug 29, 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.
Refs #1391
무엇을 · 왜
수신자 홈의 내려받기 실패 안내를 Toast → Snackbar 로 전환한다(#1391 건 1).
Toast 는 시스템 UI 라 앱 테마·시안을 안 타고, Compose semantics 에 안 잡혀 어떤 테스트로도 단언할 수 없으며 TalkBack 노출도 짧다. #664 가 이미 «repo convention = Snackbar» 로 판단한 전례가 있다.
ReceiverHomeScreen에SnackbarHostState를 세우고DownloadDialogHost가ReceiverDownloadState.Failed를 그 호스트로 흘리게 바꿨다. 문구는 기존 리소스를 그대로 쓴다.#446 이 확정한 공통 에러 팝업 4종이 아직 없으므로 잠정 Snackbar 다(#713 전례). 공통 컴포넌트가 나오면 그 체계로 재정렬되며, 그 사실은 #446 에 코멘트로 기록해 뒀다.
범위에서 뺀 것
summary/ReceiverAfternoteScreen) — 그 화면이 도달 불가임이 확인돼(수신자 summary 패키지 8파일이 도달 불가 — afternotelist 가 대체, 내려받기 전용 화면은 시안 대조 후 판정 #1424) 밑단 PR 이 파일째 삭제한다. Toast 를 Snackbar 로 바꿀 대상이 아니라 지울 대상이었다. 삭제로import android.widget.Toast는 동일하게 사라진다.MemorialReceivedDetailScreen) — 열린 PR fix(receiver): 수신 추억 노트 상세 영상 실행에 http/https 스킴 가드 이식 (#1394) #1395 가 같은 파일의 영상 실행 경로를 고치는 중이라 그 머지 후 후속.#1391은 이 건 때문에 열어 둔다(그래서Closes가 아니라Refs).검증
:feature:receiver:presentation:compileDebugKotlin·testDebugUnitTest·:app:compileDebugUnitTestKotlin·:app:compileDebugAndroidTestKotlin·ktlintCheck— 전부 BUILD SUCCESSFUL.신규
ReceiverHomeDownloadSnackbarTest가 실패 상태에서 Snackbar 문구가 뜨는 것을 Compose 로 단언한다(Toast 로는 불가능했던 검증이다).리뷰어 노트
CI Test Plan
{ "androidTest": { "mode": "none", "reason": "receiver presentation 의 수신자 홈 화면 한 곳과 그 src/test 단위 테스트만 변경한다. androidTest 소스·navigation 디렉터리·매니페스트 무접촉이라 계측 경계 변경이 없고, Snackbar 표출은 신규 Compose 단위 테스트가 단언한다." } }