Skip to content

feat(admin): 위스키 목록 지역 필터·정렬 추가 - #142

Merged
hyejj19 merged 4 commits into
mainfrom
feat/issue-463-whisky-list-filters
Sep 30, 2026
Merged

hyejj19 merged 4 commits into
mainfrom
feat/issue-463-whisky-list-filters

Conversation

@bottlenote-app

@bottlenote-app bottlenote-app Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

PR 제목 (Title)

Admin 위스키 목록 지역 필터·정렬 기준/방향 추가

변경 사항 (Changes)

  • 지역 API의 검색·페이지네이션 응답을 그대로 표시하고, 선택한 regionId로 위스키 목록을 조회합니다.
  • 한글/영문명·한글/영문 카테고리·생성일·수정일 정렬 기준과 오름/내림차순을 제공합니다.
  • 검색·필터·정렬 변경 시 첫 페이지로 돌아가며 URL·페이지 이동·뒤로가기·새로고침에서 조건을 유지합니다.
  • 정렬 API 타입 보완 및 Vitest/RTL 통합 테스트 추가.

변경 이유 (Reason for Changes)

테스트 방법 (Test Procedure)

  • pnpm exec vitest run src/pages/whisky/__tests__/WhiskyList.test.tsx src/hooks/__tests__/useRegions.test.ts src/hooks/__tests__/useAdminAlcohols.test.ts --maxWorkers=1 — 32개 통과.
  • pnpm lint — 오류 0건(기존 경고 7건); pnpm build 및 git diff --check 통과.
  • 로컬 Chrome에서 fixture 인증 상태/API 응답 주입으로 지역 선택 → 정렬 → 페이지 이동 → 뒤로가기 → 새로고침, 데스크톱·모바일 화면과 요청 파라미터 확인. 개인 계정 로그인이나 인증된 live 화면 검증은 수행하지 않았습니다.
  • 별도로 Admin dev live API에서 Agent 읽기 권한으로 지역 검색과 regionId/sortType/sortOrder를 포함한 위스키 조회가 HTTP 200인 것을 확인했습니다.

참고 사항 (Additional Information)

  • 상위·하위 지역 포함 여부는 FE에서 추론하지 않고 API 필터 결과에 맡깁니다.
  • 모바일의 기존 목록 표·페이지네이션은 좁은 화면에서 줄바꿈이 많습니다. 새 필터 조작에는 겹침/가로 넘침이 없음을 확인했으며, 기존 표 개선은 이번 변경 범위 밖입니다.
  • PR Preview는 요청되지 않아 배포 라벨을 붙이지 않습니다.

@hyejj19 hyejj19 added the preview-deploy PR 내부 테스트 배포 label Sep 30, 2026
@hyejj19
hyejj19 merged commit 4abf65d into main Sep 30, 2026
4 checks passed
@hyejj19
hyejj19 deleted the feat/issue-463-whisky-list-filters branch September 30, 2026 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview-deploy PR 내부 테스트 배포

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant