Skip to content

feat: product bgm/#662#666

Merged
Finefinee merged 9 commits into
developfrom
feat/product-bgm/#662
Apr 11, 2026
Merged

feat: product bgm/#662#666
Finefinee merged 9 commits into
developfrom
feat/product-bgm/#662

Conversation

@redjungi09

Copy link
Copy Markdown
Collaborator

변경사항

ProductCategory에 BGM 추가
Product 엔티티에 audio 필드 추가 (오디오 파일 URL, nullable)
상품 생성/수정 API 요청에 audio 필드 추가
EquippedItemsData 및 관련 DTO에 bgm 장착 아이템 필드 추가
EquippedItemsAssembler에서 BGM 카테고리 처리 추가
products 테이블 audio 컬럼 추가 및 user_equipped_items check constraint에 BGM 추가

관련 이슈

Closes #662

추가 컨텍스트

BGM은 기존 INSIGNIA / NAMEPLATE / BANNER와 달리 오디오 파일 URL(audio)이 별도로 필요해 Product 전 레이어에 걸쳐 필드를 추가했습니다.

audio 필드는 BGM 전용이며 다른 카테고리에서는 null로 내려갑니다. equippedItems.bgm은 랭킹, 프로필, 라이벌 등 기존에 장착 아이템을 반환하던 모든 API에 자동으로 포함됩니다.

@redjungi09 redjungi09 self-assigned this Apr 11, 2026
@redjungi09 redjungi09 added the enhancement 새로운 기능 추가 label Apr 11, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new product category, BGM, and adds an audio field to the product entity across the database, domain, and API layers. The changes include updating the JPA entity, DTOs, mappers, and API documentation to support background music items. Review feedback suggests enforcing data integrity by ensuring the audio field is only populated for the BGM category, increasing the database column length for audio URLs to prevent truncation, and refining test cases to use realistic data for BGM products.

Comment thread src/main/resources/db/migration/V31__add_bgm_product_category.sql
@Finefinee Finefinee merged commit d02901f into develop Apr 11, 2026
1 check passed
@Finefinee Finefinee deleted the feat/product-bgm/#662 branch April 11, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants