fix(store): 찜한 매장 카드 이미지를 상품 대표 이미지로 통일 - #217
Conversation
이슈 #216 반영. myWishlistedStores 카드의 imageUrls가 매장 대표 이미지(StoreImage)를 쓰고 있어 인기 매장 카드(PopularStore.cakeImageUrls, 상품 대표 이미지)와 소스가 달랐다 — 같은 매장이 화면마다 다른 이미지로 보이고, 매장 대표 이미지 미등록 시 카드가 비는 문제. 변경점: - StoreRepository.findStoreCakeImages에 limit 파라미터 추가(기본값 기존 4장 유지) - StoreWishlistService: StoreImage 대신 findStoreCakeImages(3장) 재사용, 평점 집계와 Promise.all 병렬화 - StoreWishlistRepository: 목록 select에서 store_images 제거 - SDL imageUrls 설명 갱신 회귀 테스트 2건 교체: 최신 상품 3장 우선·이미지 없는 매장 빈 배열 / 삭제 이미지·비활성 상품 제외. Closes #216 Claude-Session: https://claude.ai/code/session_01KfiGgWooJdsa4iPQPmqvBj
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🧹 knip — dead-code 리포트전체 리포트
|
🩺 NestJS Doctor — 89/100 (Good)진단 283건 (error 0).
architecture / security 상위 항목
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Coverage report
Test suite run success1636 tests passing in 190 suites. Report generated by 🧪jest coverage report action from 954d235 |
fix(store): 찜한 매장 카드 이미지를 상품 대표 이미지로 통일
배경
이슈 #216 반영입니다.
myWishlistedStores카드의imageUrls가 매장 대표 이미지(StoreImage)를 쓰고 있어, 인기 매장 카드(PopularStore.cakeImageUrls, 상품 대표 이미지)와 소스가 달랐습니다.같은 매장이 화면마다 다른 이미지로 보이고, 셀러가 매장 대표 이미지를 등록하지 않으면 찜 목록 카드만 비는 문제가 있어 상품 이미지 방식으로 통일합니다.
주요 결정 사항
findStoreCakeImages를 재사용하고limit파라미터만 추가.인기 매장(4장)·오늘픽업 호출부는 기본값으로 기존 동작을 유지하고, 찜 카드만 3장으로 제한합니다.
Promise.all로 병렬화.store_imagesselect는 제거했습니다(더 이상 사용하지 않음).검증
yarn validate통과 (190 suites / 1,636 tests).Closes #216