Skip to content

[feat] 위스키 카테고리 그룹 관리 기능 추가#40

Merged
hyejj19 merged 1 commit intomainfrom
feature/alcohol-category-admin
Apr 4, 2026
Merged

[feat] 위스키 카테고리 그룹 관리 기능 추가#40
hyejj19 merged 1 commit intomainfrom
feature/alcohol-category-admin

Conversation

@hyejj19
Copy link
Copy Markdown
Collaborator

@hyejj19 hyejj19 commented Apr 4, 2026

PR 제목 (Title)

위스키 카테고리를 2단계(그룹 → 서브카테고리) 선택 방식으로 개편하여 카테고리 관리 UX 개선

변경 사항 (Changes)

  • 카테고리 그룹 상수 및 매핑 함수 추가 (CATEGORY_GROUP_LABELS, getCategoryGroup, GROUP_TO_CATEGORY)
  • WhiskyBasicInfoCard: 단일 카테고리 드롭다운 → 카테고리 그룹 Select + 서브카테고리 조건부 입력 UI로 변경
  • 메인 그룹(싱글몰트, 블렌디드 등) 선택 시 한글/영문 카테고리 자동 세팅 (읽기 전용)
  • OTHER 그룹 선택 시 기존 카테고리에서 선택하거나 자유 입력 가능
  • WhiskyList: 카테고리 옵션을 중앙화된 상수에서 생성, 테이블에 카테고리 그룹 서브라벨 표시
  • CategoryReference.categoryGroup을 optional로 수정 (실제 API 응답에 포함되지 않음)
  • useCategoryReferences 훅 및 getCategoryGroup 함수 테스트 추가
  • MSW 핸들러에 카테고리 레퍼런스 엔드포인트 추가

변경 이유 (Reason for Changes)

기존에는 CategoryReference API 응답의 flat한 카테고리 목록을 그대로 드롭다운에 표시했으나, 카테고리가 많아지면서 메인 그룹(싱글몰트, 버번 등)과 기타 카테고리를 구분할 필요가 생김. 2단계 선택 방식으로 개편하여 메인 그룹은 빠르게 선택하고, 기타 카테고리는 기존 목록에서 선택하거나 자유 입력할 수 있도록 UX를 개선함.

테스트 방법 (Test Procedure)

  • pnpm test:run — 전체 136개 테스트 통과 확인
  • 위스키 상세 페이지에서 카테고리 그룹 변경 시 한글/영문 카테고리 자동 세팅 확인
  • OTHER 선택 시 기존 카테고리 선택 및 자유 입력 동작 확인
  • 위스키 목록 페이지에서 카테고리 그룹 서브라벨 정상 표시 확인

참고 사항 (Additional Information)

  • CategoryReference API가 categoryGroup 필드를 내려주지 않으므로, 프론트엔드에서 getCategoryGroup() 함수로 korCategorycategoryGroup 매핑을 수행
  • CATEGORY_GROUP_LABELS(UI 라벨)와 GROUP_TO_CATEGORY(API 값)는 의도적으로 다른 값 사용 (예: 싱글몰트 vs 싱글 몰트)

🤖 Generated with Claude Code

- WhiskyList 카테고리 서브라벨이 필터값을 표시하던 버그 수정 (getCategoryGroup 사용)
- 배럴 파일 import를 직접 모듈 import로 변경 (CLAUDE.md 준수)
- CategoryReference.categoryGroup을 optional로 수정 및 타입 단언 제거

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 4, 2026 05:09
@hyejj19 hyejj19 self-assigned this Apr 4, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

위스키 카테고리 선택 UX를 2단계(그룹 → 서브카테고리)로 확장하기 위한 기반으로, 카테고리 그룹 상수/매핑 유틸을 추가하고 목록/테스트/목 데이터를 정리한 PR입니다.

Changes:

  • 카테고리 그룹 라벨/목록 상수 및 korCategory → categoryGroup 매핑 함수(getCategoryGroup) 추가
  • 위스키 목록에서 카테고리 옵션을 상수 기반으로 생성하고, 테이블에 카테고리 그룹 서브라벨 표시
  • CategoryReference.categoryGroup을 optional로 변경하고 관련 mock/test 보강

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/types/api/alcohol.api.ts 카테고리 그룹 라벨/목록/매핑 유틸 추가 및 CategoryReference.categoryGroup optional 반영
src/test/mocks/data.ts categoryGroup 없는 실제 API 형태에 맞춘 category reference mock 데이터 추가
src/pages/whisky/WhiskyList.tsx 카테고리 옵션 생성 로직을 상수 기반으로 중앙화하고 목록에 그룹 서브라벨 표시
src/hooks/tests/useAdminAlcohols.test.ts useCategoryReferencesgetCategoryGroup에 대한 테스트 케이스 추가

@hyejj19 hyejj19 merged commit ccc14d8 into main Apr 4, 2026
6 checks passed
@hyejj19 hyejj19 deleted the feature/alcohol-category-admin branch April 4, 2026 05:17
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.

2 participants