Skip to content

#475 [Feat] 철학자 보이스 매핑에 이미지 키(image_key) 추가 - #476

Merged
si-zero merged 5 commits into
devfrom
feat/#475
Sep 12, 2026
Merged

#475 [Feat] 철학자 보이스 매핑에 이미지 키(image_key) 추가#476
si-zero merged 5 commits into
devfrom
feat/#475

Conversation

@si-zero

@si-zero si-zero commented Sep 12, 2026

Copy link
Copy Markdown
Member

#️⃣ 연관된 이슈

📝 작업 내용

✨ Feat

내용 파일
philosopher_voiceimage_key 컬럼 + 엔티티 필드 추가 PhilosopherVoice.java
요청·응답 DTO에 imageKey 추가 PhilosopherVoiceRequest.java, PhilosopherVoiceResponse.java
생성/수정 로직에 imageKey 반영 PhilosopherVoiceService.java

📌 공유 사항

  1. PhilosopherType enum(철학자 유형 10인)만 실제 이미지가 있고, 그 밖의 철학자(루소, 홉스, 쇼펜하우어 등)는 이름을 해시해서 10개 이미지 중 하나를 무작위 배정하는 폴백(PhilosopherType.resolveImageKey)이 있었습니다 — 엉뚱한 사람 이미지가 뜨는 문제. 프론트 배틀 생성 화면에서 이 랜덤 배정 대신 명시적 매핑을 쓰기 위한 필드입니다.
  2. 새 테이블을 안 만들고 기존 philosopher_voice에 컬럼만 추가했습니다 — 보이스와 이미지가 같은 시점·같은 어드민 화면에서 관리될 데이터라서입니다.
  3. image_key는 nullable. ddl-auto:update가 컬럼 자동 추가하므로 마이그레이션 불필요. 기존 58건 시드 데이터의 image_key는 비어있는 채로 유지되며, 채우는 건 별도 작업(운영/기획 쪽에서 실제 이미지 에셋 키 확정 필요).
  4. 파서(BattleScriptParseService)의 voiceSettings 자동 채움과 달리, 옵션 imageUrl 자동 채움에는 아직 연결 안 함 — 필요하면 후속 작업으로 진행 가능.

✅ 체크리스트

  • 컨벤션에 맞는 Type을 선택했나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요? (dev)

📸 스크린샷

신규 테스트 1건(imageKey 생성/수정) 추가, 전체 테스트 스위트 통과.

@si-zero si-zero self-assigned this Sep 12, 2026
@si-zero si-zero added the ✨ Feat (기능 추가) 새로운 기능을 추가할 때 사용합니다. label Sep 12, 2026
@si-zero
si-zero merged commit 58ce7ad into dev Sep 12, 2026
1 check passed
@si-zero
si-zero deleted the feat/#475 branch September 12, 2026 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feat (기능 추가) 새로운 기능을 추가할 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant