[refactor/#408] Personalization Profile 최상위 도메인 승격#414
Merged
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
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.
❤️ 기능 설명
Personalization Profile 컨텍스트를
com.techfork.domain.personalization에서 최상위 bounded context인com.techfork.personalization으로 승격했습니다.PersonalizationProfileDocument,PersonalizationProfileDocumentRepository를personalization.infrastructure로 이동PersonalizationProfileService,PersonalizationProfileEventListener를personalization.application으로 이동PersonalizationProfileScheduler를personalization.scheduler로 이동user_profiles)과 document 필드 구조는 변경하지 않음테스트 결과:
연결된 issue
close #408
🩷 Approve 하기 전 확인해주세요!
PersonalizationProfileService책임 분리([REFACTOR] Personalization Profile 생성 책임 분리 #409), 추천 트리거 이벤트 분리([REFACTOR] 프로필 생성 후 추천 트리거를 이벤트로 분리 #410), 이벤트 경계 테스트([TEST] Personalization/Recommendation 이벤트 경계 테스트 보강 #411), 문서 갱신([DOCS] Personalization Profile 문서/유비쿼터스 언어 갱신 #412)은 후속 이슈로 남깁니다.✅ 체크리스트