-
Notifications
You must be signed in to change notification settings - Fork 0
feat(Order): order 조회 칼럼 추가 #68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The head ref may contain hidden characters: "feature/#67-order\uC870\uD68C\uCE7C\uB7FC\uBCC0\uACBD"
Conversation
- 메뉴명 & 수량 추가
|
""" WalkthroughOrderResponseDto 클래스에 메뉴 이름과 수량을 집계하는 필드가 추가되고, OrderService 클래스에서는 storeId로 매장 존재 여부를 확인하는 검증 로직이 추가되었습니다. 이에 따라 관련 메서드와 의존성 주입이 수정되었습니다. Changes
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing Touches
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 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/dto/OrderResponseDto.java(2 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/service/OrderService.java(2 hunks)
🔇 Additional comments (6)
nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/service/OrderService.java (2)
12-14: 필요한 import 문들이 적절히 추가되었습니다.새로운 메뉴 검증 기능을 위한 import 문들이 올바르게 추가되었습니다.
33-33: MenuRepository 의존성 주입이 올바르게 추가되었습니다.Spring의 표준 의존성 주입 패턴을 따라 MenuRepository가 적절히 주입되었습니다.
nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/dto/OrderResponseDto.java (4)
4-8: 필요한 import 문들이 적절히 추가되었습니다.새로운 컬렉션 처리 기능을 위한 import 문들이 올바르게 추가되었습니다.
23-23: 메뉴 이름과 수량을 위한 필드가 적절히 추가되었습니다.PR 목표에 부합하는
menuNamesAndQuantities필드가 Map<String, Integer> 타입으로 적절히 정의되었습니다.
27-32: 메뉴 이름과 수량 집계 로직이 훌륭하게 구현되었습니다.Stream API를 사용하여 주문 항목들을 메뉴명별로 그룹화하고 수량을 합산하는 로직이 매우 잘 구현되었습니다. 특히
Integer::sum을 사용하여 중복 메뉴명의 수량을 합치는 처리가 우수합니다.
40-40: 빌더에 새 필드가 올바르게 추가되었습니다.새로운
menuNamesAndQuantities필드가 빌더 패턴에 적절히 추가되어 DTO 구성을 완료했습니다.
nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/service/OrderService.java
Outdated
Show resolved
Hide resolved
- 메뉴명 & 수량 추가
작업 요약
Issue Link
#67
문제점 및 어려움
해결 방안
Reference
Summary by CodeRabbit
신규 기능
버그 수정