Skip to content
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

Open
wants to merge 5 commits into
base: jaegil-jeong
Choose a base branch
from

Conversation

jaegil-jeong
Copy link

안녕하세요 리뷰어님,

너무 늦어 죄송합니다..
Step2 요구사항 정리하였습니다.
리뷰 요청드립니다.

(자꾸 이전 step의 커밋내용이 PR에 중복으로 포함되는 원인을 모르겠습다.. 제가 뭘 잘못 올린걸까요..?😢 )

Copy link

@mj950425 mj950425 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요 재길님. 고생 많으셨습니다.
다만 요구사항들이 몇가지 누락된게 보여서 코멘트 남겼는데요.
확인 부탁드릴게요.

- [ ] 상품은 0원 이상의 가격을 가져야한다.
- [ ] 상품을 등록할 수 있다.
- [ ] 상품의 가격을 변경할 수 있다.
- [ ] 상품 가격 변경 시, 상품이 속한 메뉴의 가격도 변경된다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

상품의 가격을 변경할 때 특정 상황에 메뉴가 숨겨지는 요구사항도 적어주면 좋을 것 같아요.

- [ ] 상품의 가격을 변경할 수 있다.
- [ ] 상품 가격 변경 시, 상품이 속한 메뉴의 가격도 변경된다.
- [ ] 상품 전체를 조회할 수 있다.
### 메뉴

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

메뉴의 가격에 대한 요구사항도 빠진것같아요.

- [ ] 메뉴는 이름을 가져야 한다.
- [ ] 메뉴의 이름에는 비속어를 포함시킬 수 없다.
- [ ] 메뉴는 0원 이상의 가격을 가져야한다.
- [ ] 메뉴는 노출 여부를 가져야 한다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

가격에 따른 노출이 불가능한 요구사항도 있어야할 것 같아요.

- [ ] 메뉴는 노출 여부를 가져야 한다.
- [ ] 하나의 메뉴는 여러 구성 상품을 가진다.
- [ ] 하나의 메뉴는 여러 주문에 포함될 수 있다.
### 메뉴 그룹

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

메뉴그룹만 조회하는 기능도 필요할 것 같아요.

- [ ] 메뉴 그룹은 이름을 가져야 한다.
- [ ] 하나의 메뉴 그룹에는 여러 메뉴들이 포함될 수 있다.
### 메뉴 구성 상품
- [ ] 메뉴 구성 상품은 수량을 가져야 한다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수량이 0개 이상이어야함을 명시하면 좋을 것 같네요.

- [ ] 하나의 메뉴 그룹에는 여러 메뉴들이 포함될 수 있다.
### 메뉴 구성 상품
- [ ] 메뉴 구성 상품은 수량을 가져야 한다.
### 주문

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

매장 주문의 경우 주문 항목 수량이 0개 미만일 수 있을 것 같아요.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

조회 기능에 대한 요구사항이 없네요.

- [ ] 메뉴 구성 상품은 수량을 가져야 한다.
### 주문
- [ ] 주문은 배달 주문, 포장 주문, 매장 주문 타입을 가진다.
- [ ] 주문은 주문 대기, 주문 확인, 주문 준비 완료, 주문 배달 중, 주문 배달 완료, 주문 완료 상태를 가진다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주문 타입별로 가질 수 있는 상태를 다르게 요구해야할 것 같아요.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

또한 각 타입의 주문마다 요구사항이 다를 것 같아요.

- [ ] 하나의 주문은 여러 주문 항목들을 포함할 수 있다.
### 주문 항목
- [ ] 주문 항목은 수량을 가져야 한다.
### 주문 테이블

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

손님이 없는 경우에는 매장 주문 등록이 불가능하다는 요구사항이 빠진것같아요.

@mj950425
Copy link

안녕하세요 리뷰어님,

너무 늦어 죄송합니다.. Step2 요구사항 정리하였습니다. 리뷰 요청드립니다.

(자꾸 이전 step의 커밋내용이 PR에 중복으로 포함되는 원인을 모르겠습다.. 제가 뭘 잘못 올린걸까요..?😢 )

아래 문서를 확인해보시겠어요?
https://github.com/next-step/nextstep-docs/blob/master/codereview/review-step3.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants