Open
Conversation
mj950425
requested changes
Feb 10, 2025
mj950425
left a comment
mj950425
left a comment
There was a problem hiding this comment.
안녕하세요 재길님. 고생 많으셨습니다.
다만 요구사항들이 몇가지 누락된게 보여서 코멘트 남겼는데요.
확인 부탁드릴게요.
| - [ ] 상품은 0원 이상의 가격을 가져야한다. | ||
| - [ ] 상품을 등록할 수 있다. | ||
| - [ ] 상품의 가격을 변경할 수 있다. | ||
| - [ ] 상품 가격 변경 시, 상품이 속한 메뉴의 가격도 변경된다. |
There was a problem hiding this comment.
상품의 가격을 변경할 때 특정 상황에 메뉴가 숨겨지는 요구사항도 적어주면 좋을 것 같아요.
| - [ ] 상품의 가격을 변경할 수 있다. | ||
| - [ ] 상품 가격 변경 시, 상품이 속한 메뉴의 가격도 변경된다. | ||
| - [ ] 상품 전체를 조회할 수 있다. | ||
| ### 메뉴 |
| - [ ] 메뉴는 이름을 가져야 한다. | ||
| - [ ] 메뉴의 이름에는 비속어를 포함시킬 수 없다. | ||
| - [ ] 메뉴는 0원 이상의 가격을 가져야한다. | ||
| - [ ] 메뉴는 노출 여부를 가져야 한다. |
| - [ ] 메뉴는 노출 여부를 가져야 한다. | ||
| - [ ] 하나의 메뉴는 여러 구성 상품을 가진다. | ||
| - [ ] 하나의 메뉴는 여러 주문에 포함될 수 있다. | ||
| ### 메뉴 그룹 |
| - [ ] 메뉴 그룹은 이름을 가져야 한다. | ||
| - [ ] 하나의 메뉴 그룹에는 여러 메뉴들이 포함될 수 있다. | ||
| ### 메뉴 구성 상품 | ||
| - [ ] 메뉴 구성 상품은 수량을 가져야 한다. |
| - [ ] 하나의 메뉴 그룹에는 여러 메뉴들이 포함될 수 있다. | ||
| ### 메뉴 구성 상품 | ||
| - [ ] 메뉴 구성 상품은 수량을 가져야 한다. | ||
| ### 주문 |
There was a problem hiding this comment.
매장 주문의 경우 주문 항목 수량이 0개 미만일 수 있을 것 같아요.
| - [ ] 메뉴 구성 상품은 수량을 가져야 한다. | ||
| ### 주문 | ||
| - [ ] 주문은 배달 주문, 포장 주문, 매장 주문 타입을 가진다. | ||
| - [ ] 주문은 주문 대기, 주문 확인, 주문 준비 완료, 주문 배달 중, 주문 배달 완료, 주문 완료 상태를 가진다. |
| - [ ] 하나의 주문은 여러 주문 항목들을 포함할 수 있다. | ||
| ### 주문 항목 | ||
| - [ ] 주문 항목은 수량을 가져야 한다. | ||
| ### 주문 테이블 |
There was a problem hiding this comment.
손님이 없는 경우에는 매장 주문 등록이 불가능하다는 요구사항이 빠진것같아요.
아래 문서를 확인해보시겠어요? |
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
2 participants
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.
안녕하세요 리뷰어님,
너무 늦어 죄송합니다..
Step2 요구사항 정리하였습니다.
리뷰 요청드립니다.
(자꾸 이전 step의 커밋내용이 PR에 중복으로 포함되는 원인을 모르겠습다.. 제가 뭘 잘못 올린걸까요..?😢 )