Skip to content

feat: add curation feed search filters#656

Merged
Whale0928 merged 3 commits into
mainfrom
feat/curation-feed-search
Jul 2, 2026
Merged

feat: add curation feed search filters#656
Whale0928 merged 3 commits into
mainfrom
feat/curation-feed-search

Conversation

@bottlenote-app

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

Copy link
Copy Markdown
Contributor

Change Log

Added

  • GET /api/v2/curations/feedkeyword, code query parameter를 추가했습니다.
  • keyword는 단일 문자열로 큐레이션 제목/설명, 스펙 제목/설명 대상 부분 일치 검색을 수행합니다.
  • codekeyword와 분리된 스펙 코드 exact match 필터로 동작합니다.

Changed

  • Feed 조회 시 필터 적용 후 cursor pagination이 수행되도록 조회 흐름을 확장했습니다.
  • blank keyword/code는 필터 미적용 조건으로 처리합니다.
  • API 문서에 요청 파라미터와 code 확인 방법을 추가했습니다.

Tests

  • Product feed service 테스트에 positive/negative/boundary 케이스를 추가했습니다.
  • REST Docs 테스트에 feed query parameter 문서화를 추가했습니다.
  • Product API 통합 테스트에 keyword/code 필터 케이스를 확장했습니다.

Verification

  • ./gradlew :bottlenote-mono:test --tests 'app.bottlenote.curation.service.ProductSpecBasedCurationServiceTest'
  • ./gradlew :bottlenote-product-api:test --tests 'app.docs.curation.RestProductSpecBasedCurationControllerTest'
  • ./gradlew :bottlenote-product-api:asciidoctor

Notes

  • 로컬 통합 테스트 단독 실행은 Docker/Testcontainers 환경 초기화 단계에서 실패하여 assertion 진입 전 중단되었습니다.

Related Workspace Issue

@Whale0928 Whale0928 merged commit 1acb357 into main Jul 2, 2026
7 checks passed
@Whale0928 Whale0928 deleted the feat/curation-feed-search branch July 2, 2026 12:02
@bottlenote-app

bottlenote-app Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Workspace 이슈 연결 완료: bottle-note/workspace#294

@bottlenote-app

bottlenote-app Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

@HyeJungPark 개발 서버 배포 및 Product 큐레이션 feed API 동작 확인했습니다.

검증 요약:

  • code=WHISKY_TASTING_EVENT 기준 코드별 피드 조회 및 cursor 페이지 동작 확인
  • keyword 단독, keyword+code 조합, 알 수 없는 code, 공백 조건 처리 확인
  • feed payload는 x-feed.enabled=true 대상 필드만 포함되는 것으로 확인

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.

1 participant