Refactor: isWaiting 필드 추가#163
Conversation
|
Caution Review failedThe pull request is closed. """ Walkthrough이번 변경은 매장 상세 정보를 반환하는 API 응답 구조를 개선하고, 사용자의 대기 여부를 Redis에서 확인하는 기능을 추가합니다. 주요 변경점은 새로운 DTO 도입, 서비스 계층의 반환 타입 변경, 대기 여부 확인 메서드 추가, 그리고 관련된 데이터 변환 로직의 확장입니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Controller
participant StoreServiceImpl
participant WaitingUserRedisRepository
participant RedisTemplate
Controller->>StoreServiceImpl: getStoreByStoreId(storeId, user)
StoreServiceImpl->>WaitingUserRedisRepository: isUserWaiting(storeId, userId)
WaitingUserRedisRepository->>RedisTemplate: opsForZSet().rank(key, userId)
WaitingUserRedisRepository-->>StoreServiceImpl: Boolean (isWaiting)
StoreServiceImpl->>RedisTemplate: opsForZSet().zCard(key)
StoreServiceImpl-->>Controller: StoreDetailReadResponse (with isWaiting, waitingCount, etc.)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~15–20 minutes Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (5)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
작업 요약
사용자의 대기 상태 확인을 위한 isWaiting 필드 추가
Issue Link
#154
문제점 및 어려움
해결 방안
Reference
Summary by CodeRabbit
신규 기능
개선 사항
버그 수정