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 요구 사항 정리 #859

Open
wants to merge 5 commits into
base: reversalspring
Choose a base branch
from

Conversation

reversalSpring
Copy link

안녕하세요? step2 요구사항 정리 리뷰 요청 드립니다.

Copy link

@sihoony sihoony left a comment

Choose a reason for hiding this comment

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

안녕하세요! 준형님,
간단한 코멘트들은 남겼어요.
개발을 진행하기전에 어떻게 정의되어 전달되면 좋을지 고민해보고 작성해보면 어떨까요~! 😄
작성 방법에 대한 방향이 잡힐 것 같네요~! 즐거운 하루 되세요~! 😄

- 테이블이 비어있는지, 사용 중인지 설정할 수 있다.
- [ ] 주문 테이블 목록을 조회할 수 있다.

### 주문 관리
Copy link

Choose a reason for hiding this comment

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

주문 상태에 대한 정의를 별도로 해봐도 좋을 것 같아요~! 😄

@@ -9,6 +9,46 @@ docker compose -p kitchenpos up -d

## 요구 사항

### 상품 관리
- [ ] 상품을 등록할 수 있다.
- 상품명과 가격 정보가 포함되어야 한다.
Copy link

Choose a reason for hiding this comment

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

가격에 대한 제약이 무엇이있을까요? 마이너스 가격이 들어가도 되는지 명확한 요구사항을 정의하면 좋을 것 같아요~!

- [ ] 메뉴를 등록할 수 있다.
- 메뉴명, 가격, 메뉴 그룹, 출력 여부가 포함되어야 한다.
- 메뉴에 포함되는 상품 정보를 입력할 수 있다.
- [ ] 메뉴 가격을 변경할 수 있다.
Copy link

Choose a reason for hiding this comment

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

가격 변동은 어디까지 허용하는 걸까요~! 😄

Copy link

Choose a reason for hiding this comment

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

가격 변경에 대한 요구사항도 추가해보면 좋을 것 같아요.

README.md Outdated
Comment on lines 27 to 28
- [ ] 메뉴 출력 여부를 설정할 수 있다.
- 메뉴를 숨기거나 보이게 할 수 있다.
Copy link

Choose a reason for hiding this comment

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

메뉴를 숨기거나 보여지게하는 조건은 무엇이 있을까요~!?

Copy link

@sihoony sihoony left a comment

Choose a reason for hiding this comment

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

안녕하세요~! 준형님,
간단한 코멘트 남겼습니다. 😄
코드와 http 파일을 따라서 어떤 요구사항들이 포함되어야 하는지 작성해보면 좋을 것 같아요~!
즐거운 하루 되세요~! 😄

Comment on lines +16 to +17
- 가격은 100원 단위로 설정 가능하다.
- 최대 가격은 1,000,000원으로 제한한다.
Copy link

Choose a reason for hiding this comment

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

가격 정책에 대한 단위설정, 최대치가 코드상에 있는지 확인해보면 좋을 것 같아요~! 😄

- [ ] 메뉴를 등록할 수 있다.
- 메뉴명, 가격, 메뉴 그룹, 출력 여부가 포함되어야 한다.
- 메뉴에 포함되는 상품 정보를 입력할 수 있다.
- [ ] 메뉴 가격을 변경할 수 있다.
Copy link

Choose a reason for hiding this comment

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

가격 변경에 대한 요구사항도 추가해보면 좋을 것 같아요.

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