Skip to content

refactor(afternote): 에디터 곡·영상 변환 규칙을 공통화한다 (#2166) - #2167

Merged
OnlyReadOneBook merged 5 commits into
developfrom
refactor/2166-media-mapping
Oct 4, 2026
Merged

OnlyReadOneBook merged 5 commits into
developfrom
refactor/2166-media-mapping

Conversation

@OnlyReadOneBook

@OnlyReadOneBook OnlyReadOneBook commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Closes #2166
Refs #2163

생성 요청과 현재값 스냅샷에 반복되던 곡 객체 조립, 영상 URL·썸네일 정규화를 폼 매퍼의 private 함수로 모읍니다. 유효한 영상 URL과 빈 썸네일의 null 처리, 곡 값과 순서, DTO·PATCH 동작은 유지합니다.

직접 base는 #2164의 refactor/2163-category-snapshot입니다. 이 변환 구조를 #1837 이후 에디터 스택이 그대로 소비합니다. 최신 develop의 영상 세 상태와 스냅샷 v6를 포함합니다.

검증

HEAD f67d10e59b6802036f6314f38fe1e78fab0d7456에서 presentation 340개 테스트가 통과했습니다(실패·오류·건너뜀 0). presentation ktlint, 실제 base 대비 전체 diff의 모듈 소유권 검사와 git diff --check를 통과했습니다.

🧪 CI Test Plan

{
  "androidTest": {
    "mode": "none",
    "reason": "같은 파일 안의 순수 변환 코드 추출이며 UI·Android API 변경 없이 기존 생성 및 PATCH 단위 테스트로 검증합니다.",
    "tests": []
  }
}

@Sadturtleman Sadturtleman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

곡·영상 변환을 private 함수로 모으면서 빈 값 처리와 곡 순서가 그대로 유지되는 것을 확인했습니다. 승인합니다.

@OnlyReadOneBook
OnlyReadOneBook added this pull request to the merge queue Oct 4, 2026
Merged via the queue into develop with commit ca1c18d Oct 4, 2026
23 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the refactor/2166-media-mapping branch October 4, 2026 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(afternote): 에디터 곡·영상 변환 규칙을 공통화한다

3 participants