Repository navigation
feat: 사장님 주문 상태 변경 API 추가 - #2443
Merged
Soundbar91 merged 9 commits intoSep 22, 2026
Merged
Soundbar91 merged 9 commits into
Soundbar91 merged 9 commits into
Conversation
POS 화면의 승인, 반려, 조리 완료, 배달 완료 버튼을 처리한다. 전이마다 엔드포인트를 두는 대신 목표 상태를 요청 바디로 받아 하나로 합쳤다. 허용 전이를 OrderStatus가 알도록 하여 어느 경로로 들어오든 동일하게 막는다. 반려는 결제 전액 취소까지 함께 수행한다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
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: defaults Review profile: CHILL Plan: Advanced 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 |
POS 화면의 영업 시작, 영업 종료 버튼을 처리한다. shop_operation.is_open 컬럼은 있었으나 이를 변경하는 코드가 없어 사장님이 직접 영업을 여닫을 방법이 없었다. 주문 가능 상점으로 설정되지 않은 상점은 변경할 수 없다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
POS의 다른 API가 모두 orderableShopId를 받는데 영업 토글만 shopId를 받아 클라이언트가 식별자 두 개를 관리해야 했다. 주문 가능 상점 기준으로 받고 컨트롤러와 서비스도 주문 도메인에 둔다. 연관관계 탐색은 OrderableShop.changeOpenStatus 안에 가둔다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
3 of 4 tasks
/owner/shops/{id}는 shopId를 받는데 POS API는 같은 자리에서
orderableShopId를 받아, 다른 종류의 식별자가 한 경로에 섞여 있었다.
둘 다 작은 정수라 잘못 넣어도 조용히 다른 가게가 조회될 수 있다.
고객용 /order/shop/{orderableShopId}와 같은 형태로 맞춰 구분한다.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
POS 로그인 후 매장 선택 화면에 필요한 목록을 반환한다. 기존 /owner/shops는 shopId와 이름만 내려주어 이후 POS API에 필요한 orderable_shop_id와 주소, 영업 여부를 얻을 수 없었다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
목록과 상세 응답에 order_type을 추가한다. 기존 구현은 배달 정보를 널 검사 없이 꺼내 포장 주문이 들어오면 터졌고, 완료 탭에 포장 완료와 포장 수령이 빠져 있어 포장 주문이 조리중 이후 어느 탭에도 보이지 않았다. 유형별 연관관계 탐색을 Order 안으로 옮기고, 각 유형이 가질 수 있는 상태를 OrderType이 알도록 하여 유형에 맞지 않는 전이를 막는다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…onse feat: 주문 응답에 배달/포장 유형 추가하고 포장 주문 처리
feat: 사장님 전용 주문 가능상점 조회 API 추가
feat: 가게 영업 상태 변경 API 추가
Soundbar91
merged commit Sep 22, 2026
0563688
into
feature/2438-owner-order-list-detail
4 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔍 개요
배달 전용 POS 화면에서 사장님이 주문을 승인, 반려, 조리 완료, 배달 완료 처리할 수 있도록 상태 변경 API를 구현합니다.
close [비즈니스] 주문 상태 전의 메소드 구현 #2442
🚀 주요 변경 내용
💬 참고 사항
✅ Checklist (완료 조건)