Repository navigation
fix(afternote): 업로드 크기 초과를 저장 오류와 구분한다 (#1868) - #2119
Merged
Merged
Conversation
OnlyReadOneBook
added this pull request to stack #2121
September 20, 2026 11:53
Collaborator
Author
|
실기기에서 확인했다. 2026-09-23 15:04 KST, 갤럭시 S25(SM-S931N, Android 16, 1080x2340 @420dpi, font_scale 0.8), 이 PR head
|
This was referenced Sep 23, 2026
Sadturtleman
approved these changes
Sep 25, 2026
OnlyReadOneBook
removed this pull request from stack #2121
September 25, 2026 08:53
OnlyReadOneBook
added this pull request to stack #2165
September 25, 2026 08:53
OnlyReadOneBook
removed this pull request from stack #2165
September 27, 2026 04:55
Contributor
|
이 PR 은
|
…ix/1868-upload-size-feedback # Conflicts: # feature/afternote/presentation/src/main/kotlin/com/afternote/feature/afternote/presentation/editor/AfternoteEditorViewModel.kt
Contributor
|
형제 충돌이 해소됐다 — 지금은 이 PR 과 같은 파일을 서로 다르게 고친 열린 PR 이 없다 (#1750). |
Sadturtleman
approved these changes
Oct 6, 2026
8 tasks
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.

Closes #1868
사진이나 영상 업로드가 서버의 크기 초과로 거절되면 일반 저장 실패와 구분하여 기존 크기 초과 안내를 표시합니다.
FileUploadFailure.FileSizeExceeded를 실제 Save Intent·UseCase 경로에서 소비하며 저장 요청 중단·로딩 해제·선택한 파일 보존과 일반 오류·썸네일 재시도 동작을 유지합니다.직접 base는 #1972의
refactor/1804-editor-mvi입니다. 크기 초과에는 재시도 액션을 붙이지 않습니다. 변경 범위는 afternote 저장 오류 분류와 기존 스낵바 연결 6개 파일이며 압축·리사이즈 또는 미확정 임시저장 화면의 후속 구현을 예약하지 않습니다.검증
JVM 검증 리비전
32813e93206099ef8736b986612f0c081bf34deb에서 presentation 349개 테스트가 통과했습니다(실패·오류·건너뜀 0). presentation ktlint, 실제 base 대비 전체 diff의 모듈 소유권 검사와git diff --check를 통과했습니다.현재 HEAD
47ccb020b2eb14aa6c8f2836e734240dd0bae258는 검증 리비전에서 수신자 복귀 KDoc만 최종 Intent 이름으로 정정했습니다. 프로덕션 코드와 테스트 코드는 동일하며 KDoc 링크 검사(끊김 0)와 ktlint를 통과했습니다.🧪 CI Test Plan
{ "androidTest": { "mode": "none", "reason": "기존 저장 실패의 도메인 타입 판정과 스낵바 문자열 연결만 변경합니다. 사진·영상 실패의 실제 Save Intent/UseCase 경로, 저장 중단·선택 보존·재시도 액션 제외는 Robolectric 단위 테스트로 검증하며 Android 내비게이션·권한·파일 업로드 런타임 배선은 변경하지 않습니다.", "tests": [] } }