Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
주형님, 안녕하세요!
주문 BC 하위에 공통, 매장/포장/배달 주문 애그리거트로 구성되어 있고,
단일 테이블 상속 매핑전략을 통해 주문(상위) 도메인 -> 매장/포장/배달 주문(하위) 도메인으로 구조로 설계했습니다.
공통 애그리거트 애플리케이션 계층의 facade 컴포넌트에서 하위 도메인 호출해서 사용하려고 했어요!
가장 복잡도 있는 바운디드 컨텍스트여서 시간이 꽤 소요된 것 같아요. 🫠
이번 마지막 미션 진행하면서 전반적으로 코드 리팩토링을 해보아야겠다고 다짐했고
리뷰어님이 피드백과 개념적으로 부족한 부분은 찾아보고 진행하고해서 점점 더 유지보수하기 쉬운 코드로 진행되고 있다고 생각이 들어요.
3단계 - 리팩터링(매장 식사 주문) PR 올립니다.
공식적인 리뷰요청은 끝났지만 기타 의견이나 참고사항 있으시면 피드백 부탁드려요 ! 감사합니다. 😊