-
Notifications
You must be signed in to change notification settings - Fork 0
feat(Order): order 틀 생성 #49
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: "feat/#47_\uAD00\uB9AC\uC790\uD398\uC774\uC9C0\uC8FC\uBB38\uC870\uD68C"
Conversation
Walkthrough이 변경사항은 두 개의 OrderController 클래스에서 코드 포맷팅만 수정하였으며, 하나는 빈 줄을 추가하고 다른 하나는 불필요한 빈 줄을 제거했습니다. 코드의 기능이나 로직에는 어떠한 영향도 주지 않았습니다. Changes
Possibly related PRs
Suggested reviewers
✨ 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: 0
🔭 Outside diff range comments (1)
application-admin/src/main/java/com/nowait/applicationadmin/order/controller/OrderController.java (1)
12-12:@RequestMapping경로에 이중 슬래시가 포함되어 있습니다.
admin//orders는 의도치 않은 404/301 문제를 유발할 수 있습니다. 한 개의 슬래시로 정정해 주세요.-@RequestMapping("admin//orders") +@RequestMapping("admin/orders")
🧹 Nitpick comments (1)
application-admin/src/main/java/com/nowait/applicationadmin/order/controller/OrderController.java (1)
16-19: TODO만 존재하여 API가 동작하지 않습니다.컨트롤러가 실제 주문 목록을 반환하도록 구현하거나, 아직 미정이라면 주석 대신
UnsupportedOperationException을 던져 호출 측에서 미구현임을 명확히 알 수 있도록 처리하세요.필요하시면 기본 구현 예시를 제공해 드리겠습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
application-admin/src/main/java/com/nowait/applicationadmin/order/controller/OrderController.java(1 hunks)application-user/src/main/java/com/nowait/applicationuser/order/controller/OrderController.java(0 hunks)
💤 Files with no reviewable changes (1)
- application-user/src/main/java/com/nowait/applicationuser/order/controller/OrderController.java
작업 요약
Issue Link
문제점 및 어려움
해결 방안
Reference
Summary by CodeRabbit