Skip to content

fix: Product API에서 삭제 위스키 제외#657

Open
bottlenote-app[bot] wants to merge 1 commit into
mainfrom
fix/219-hide-deleted-whisky-product-api
Open

fix: Product API에서 삭제 위스키 제외#657
bottlenote-app[bot] wants to merge 1 commit into
mainfrom
fix/219-hide-deleted-whisky-product-api

Conversation

@bottlenote-app

@bottlenote-app bottlenote-app Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

개요

  • workspace #219 대응: Admin에서 삭제 처리한 위스키가 Product API에 노출되지 않도록 Product read-path에 soft-delete 필터를 보강했습니다.
  • Admin 조회 경로는 관리 목적상 삭제 데이터 확인 가능성을 유지했습니다.

변경 내용

  • Product 검색/상세/둘러보기/카테고리 조회에서 deletedAt is null 기준 적용
  • Product용 alcohol summary/facade 조회와 existsByAlcoholId에서도 삭제 데이터 제외
  • Product 상세 조회에서 삭제/미존재 데이터는 ALCOHOL_NOT_FOUND로 처리
  • Admin 검색의 includeDeleted 및 Admin 상세 조회는 삭제 데이터 확인 가능하도록 유지

검증

  • 성공: ./gradlew :bottlenote-mono:compileJava :bottlenote-product-api:compileTestJava
  • 성공: ./gradlew :bottlenote-mono:test --tests app.bottlenote.alcohols.service.DefaultAlcoholFacadeTest
  • 로컬 제약: ./gradlew :bottlenote-product-api:integration_test --tests ...는 Testcontainers Docker client 초기화 실패로 실행 환경에서 실패했습니다. 실패 지점은 Spring context의 DockerClientProviderStrategy 초기화이며, 테스트 assertion 실패는 확인되지 않았습니다.

관련

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.

0 participants