-
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
2단계 - 요구 사항 정리 #854
base: sunggyun-jo
Are you sure you want to change the base?
2단계 - 요구 사항 정리 #854
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.
안녕하세요 성균님 :)
미션 잘 해주셨어요 👍
몇 가지 코멘트 남겼습니다~
확인하고 다시 요청해 주세요 🙇
README.md
Outdated
@@ -9,6 +9,33 @@ docker compose -p kitchenpos up -d | |||
|
|||
## 요구 사항 | |||
|
|||
- 간단한 키친포스를 구현한다. | |||
- 상품 | |||
- [ ] 상품을 등록 할 수 있다. |
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.
제약 조건들 추가해보았습니다.
README.md
Outdated
- 간단한 키친포스를 구현한다. | ||
- 상품 | ||
- [ ] 상품을 등록 할 수 있다. | ||
- [ ] 등록한 상품의 가격을 수정 할 수 있다. |
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.
수정시 예외 발생 조건들 추가 하였습니다.
README.md
Outdated
- [ ] 상품을 등록 할 수 있다. | ||
- [ ] 등록한 상품의 가격을 수정 할 수 있다. | ||
- [ ] 등록한 상품 목록을 조회 할 수 있다. | ||
- 메뉴 |
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.
메뉴 등록 조건과 상태 변경 시 예외 조건들 추가 하였습니다.
README.md
Outdated
- [ ] 메뉴 그룹을 등록 할 수 있다. | ||
- [ ] 메뉴 그룹을 조회 할 수 있다. | ||
- [ ] 메뉴를 등록 할 수 있다. 메뉴는 하나의 메뉴 그룹에 속해야 한다. 메뉴는 1개 이상의 상품으로 구성할 수 있다. | ||
- [ ] 메뉴의 가격을 수정할 수 있다. | ||
- [ ] 메뉴를 보이기/숨기기 처리 할 수 있다. |
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.
생성, 수정시 조건들과 예외 발생 조건 추가 하였습니다.
- [ ] 주문 테이블을 사용 가능 상태로 수정 할 수 있다. | ||
- [ ] 주문 테이블에 손님 숫자를 수정 할 수 있다. | ||
- [ ] 주문 테이블 목록을 조회 할 수 있다. | ||
- 주문 |
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.
주문에 더 많은 정보가 있을 것 같은데요! 확인하고 더 추가해보면 좋을 것 같아요 :)
주문 아이템 or 주문항목
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.
주문 속성을 추가했고, 각 주문 유형, 주문 상태등 도 추가로 정리하였습니다.
@liquidjoo PR 피드백 주셔서 감사합니다. 피드백 반영해서 내용 보완 하였습니다. 시간 되실 때 리뷰 부탁 드립니다. |
No description provided.