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

1단계 - 용어 사전 만들기 #482

Open
wants to merge 1 commit into
base: reversalspring
Choose a base branch
from

Conversation

reversalSpring
Copy link

늦어서 죄송합니다.

리뷰 부탁 드립니다.

Copy link

@hyunssooo hyunssooo left a comment

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.

context를 나눠주셨으면 테이블을 아예 분리해 정리해도 좋을 것 같아요. :)

Comment on lines +106 to +107
| [메뉴] |
| 메뉴 | Menu | 1개 이상의 상품으로 구성된 판매 단위. 가격은 구성 상품들의 가격 합보다 크거나 같아야 함 |

Choose a reason for hiding this comment

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

MenuProduct 도 정리해 보면 어떨까요? 실제로 상품으로 구성된 단위는 MenuProduct로 보여서요!

| 주문 | Order | 고객이 요청한 메뉴들의 집합. 배달/포장/매장 등의 주문 유형을 가짐 |
| 주문 항목 | Order Item | 주문에 포함된 개별 메뉴와 그 수량 |
| 주문 유형 | Order Type | 배달/포장/매장으로 구분되는 주문의 종류 |
| 주문 상태 | Order Status | 접수 대기/접수/서빙/배달 중/배달 완료/완료 등 주문의 처리 단계 |

Choose a reason for hiding this comment

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

Status는 각 유형별로 어떻게 변하는지 정리하는 것이 어떨까요?

Comment on lines +115 to +116
| 주문 항목 | Order Item | 주문에 포함된 개별 메뉴와 그 수량 |
| 주문 유형 | Order Type | 배달/포장/매장으로 구분되는 주문의 종류 |

Choose a reason for hiding this comment

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

주문 항목에는 메뉴가 포함되는데요. 해당 메뉴라는 용어는 메뉴 context에서의 메뉴와 동일한 의미를 가질까요? 고민해 보시면 좋을 것 같아요. :)

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