-
Notifications
You must be signed in to change notification settings - Fork 227
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
Step2 요구사항 정리 #848
base: jaegil-jeong
Are you sure you want to change the base?
Step2 요구사항 정리 #848
Conversation
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.
안녕하세요 재길님. 고생 많으셨습니다.
다만 요구사항들이 몇가지 누락된게 보여서 코멘트 남겼는데요.
확인 부탁드릴게요.
- [ ] 상품은 0원 이상의 가격을 가져야한다. | ||
- [ ] 상품을 등록할 수 있다. | ||
- [ ] 상품의 가격을 변경할 수 있다. | ||
- [ ] 상품 가격 변경 시, 상품이 속한 메뉴의 가격도 변경된다. |
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.
상품의 가격을 변경할 때 특정 상황에 메뉴가 숨겨지는 요구사항도 적어주면 좋을 것 같아요.
- [ ] 상품의 가격을 변경할 수 있다. | ||
- [ ] 상품 가격 변경 시, 상품이 속한 메뉴의 가격도 변경된다. | ||
- [ ] 상품 전체를 조회할 수 있다. | ||
### 메뉴 |
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.
메뉴의 가격에 대한 요구사항도 빠진것같아요.
- [ ] 메뉴는 이름을 가져야 한다. | ||
- [ ] 메뉴의 이름에는 비속어를 포함시킬 수 없다. | ||
- [ ] 메뉴는 0원 이상의 가격을 가져야한다. | ||
- [ ] 메뉴는 노출 여부를 가져야 한다. |
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.
가격에 따른 노출이 불가능한 요구사항도 있어야할 것 같아요.
- [ ] 메뉴는 노출 여부를 가져야 한다. | ||
- [ ] 하나의 메뉴는 여러 구성 상품을 가진다. | ||
- [ ] 하나의 메뉴는 여러 주문에 포함될 수 있다. | ||
### 메뉴 그룹 |
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.
메뉴그룹만 조회하는 기능도 필요할 것 같아요.
- [ ] 메뉴 그룹은 이름을 가져야 한다. | ||
- [ ] 하나의 메뉴 그룹에는 여러 메뉴들이 포함될 수 있다. | ||
### 메뉴 구성 상품 | ||
- [ ] 메뉴 구성 상품은 수량을 가져야 한다. |
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.
수량이 0개 이상이어야함을 명시하면 좋을 것 같네요.
- [ ] 하나의 메뉴 그룹에는 여러 메뉴들이 포함될 수 있다. | ||
### 메뉴 구성 상품 | ||
- [ ] 메뉴 구성 상품은 수량을 가져야 한다. | ||
### 주문 |
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.
매장 주문의 경우 주문 항목 수량이 0개 미만일 수 있을 것 같아요.
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.
조회 기능에 대한 요구사항이 없네요.
- [ ] 메뉴 구성 상품은 수량을 가져야 한다. | ||
### 주문 | ||
- [ ] 주문은 배달 주문, 포장 주문, 매장 주문 타입을 가진다. | ||
- [ ] 주문은 주문 대기, 주문 확인, 주문 준비 완료, 주문 배달 중, 주문 배달 완료, 주문 완료 상태를 가진다. |
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.
주문 타입별로 가질 수 있는 상태를 다르게 요구해야할 것 같아요.
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.
또한 각 타입의 주문마다 요구사항이 다를 것 같아요.
- [ ] 하나의 주문은 여러 주문 항목들을 포함할 수 있다. | ||
### 주문 항목 | ||
- [ ] 주문 항목은 수량을 가져야 한다. | ||
### 주문 테이블 |
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.
손님이 없는 경우에는 매장 주문 등록이 불가능하다는 요구사항이 빠진것같아요.
아래 문서를 확인해보시겠어요? |
안녕하세요 리뷰어님,
너무 늦어 죄송합니다..
Step2 요구사항 정리하였습니다.
리뷰 요청드립니다.
(자꾸 이전 step의 커밋내용이 PR에 중복으로 포함되는 원인을 모르겠습다.. 제가 뭘 잘못 올린걸까요..?😢 )