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

3단계 - 기능 우선 패키지 구성하기 #329

Open
wants to merge 3 commits into
base: seungjoopet
Choose a base branch
from

Conversation

seungjoopet
Copy link

@seungjoopet seungjoopet commented Sep 12, 2023

#320 리뷰를 토대로 재구성하였습니다.

  • product는 menu로부터 분리하여 다른 헥사고날 경계로 분리하였습니다.
  • eatinorder 는 매장식사라는 eatin으로 변경하고, 하위에 매장주문/매장테이블을 가지고 있게 하였습니다.
    • eatin은 eatinorder, eatintable 두 개의 aggregate을 갖지만 같은 비즈니스 경계라고 판단하여 헥사고날로 분리하지 않았습니다.

질문이 엄청 많았는데, 꼼꼼하게 리뷰해주셔서 감사합니다 😄👍

@seungjoopet
Copy link
Author

리뷰어님 제가 실수하고 있었습니다,

out/ 이 gitignore 설정되어서 그동안 ignored 처리가 되고 있더라구요^_ㅠ
처리해서 다시 올렸습니다

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.

1 participant