Repository navigation
refactor(afternote): 임시저장 프리필 계약의 이름과 매퍼 모양을 정리한다 (#2115) - #2117
Merged
Merged
Conversation
DraftDetail → DraftPrefill. 임시저장에는 상세 화면이 없고 소비처는 에디터 프리필뿐이라 데이터 출처가 아니라 용도로 이름을 짓는다. credentials 매퍼는 AfternoteCredentialsDto.toDomain() 하나로 합치고, MemorialMedia 조립은 AfternotePlaylistDto.toMemorialMedia() 한 곳으로 모은다.
Contributor
|
형제 충돌이 해소됐다 — 지금은 이 PR 과 같은 파일을 서로 다르게 고친 열린 PR 이 없다 (#1750). |
This was referenced Sep 20, 2026
Sadturtleman
approved these changes
Sep 25, 2026
Contributor
|
이 PR 은
|
…t-prefill-mapper # Conflicts: # feature/afternote/domain/src/testFixtures/kotlin/com/afternote/feature/afternote/domain/testing/FakeAfternoteRepository.kt
OnlyReadOneBook
commented
Sep 26, 2026
21 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 #2115
이미 제공된 임시저장 조회 계약의 이름을
DraftDetail/getDraftDetail에서DraftPrefill/getDraftPrefill로 맞추고, credentials와 추억 미디어 변환을 실제 값 객체의 확장 함수로 정리합니다. 직접 base는 develop입니다.기존
AfternoteRepositoryImpl.getDraftPrefill이toDraftPrefill을 호출하며 발행 상세와 임시저장 매핑이 같은 credentials·playlist 변환을 소비합니다. 기존 서버 필드, wire 형식, null 처리와 공개 저장소 계약의 기능은 유지합니다. 새 wrapper나 UI 진입점은 추가하지 않습니다.현재 조회 계약을 호출하는 프로덕션 UI/ViewModel은 없습니다. 에디터 이어쓰기 기능이 제공된다는 뜻으로 해석하지 않습니다. 미확정 기능 PR #1768과 이슈 #808은 구현 완료가 아닌 철회이며, 이 PR에는 해당 기능의 후속 구현 예약이 없습니다.
검증
HEAD
bd0b84a56d736dfb2105e67ae843aa695d2ee4f3에서 presentation 333개, domain 11개, data 98개 테스트가 통과했습니다(실패·오류·건너뜀 0). presentation ktlint, 실제 base 대비 전체 diff의 모듈 소유권 검사와git diff --check를 통과했습니다. domain/data ktlint도 통과했습니다.🧪 CI Test Plan
{ "androidTest": { "mode": "none", "reason": "domain 모델·repository 계약과 data 매퍼·구현, 그 단위 테스트와 testFixtures 만 바꿨다. presentation 의 navigation 패키지·AndroidManifest·MainActivity·androidTest 소스셋·Gradle 파일을 건드리지 않아 계측 경계가 움직이지 않는다.", "tests": [] } }