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.

Sign up for GitHub

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

Jump to bottom

Feature 69 add student or manager created event #76

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

Feature 69 add student or manager created event #76

wants to merge 3 commits into from

Conversation

enrouteeee
Copy link
Member

close #69

@seongbin9786
Copy link
Member

이 PR은 도메인 이벤트에 대한 공통된 이해를 구축하고 넘어가는 것이 어떠신가용

@seongbin9786
Copy link
Member

도메인 이벤트를 발행하는 것은 애그리거트, 엔터티의 역할이고, 트랜잭션에 aop 훅을 걸어서 changes를 publish하는 형태로 구현하는 게 일반적인 것 같습니다. (저번에 관련한 내용을 채널에 올렸습니다. )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arsgsg1 arsgsg1 Awaiting requested review from arsgsg1 arsgsg1 is a code owner

@Devconf Devconf Awaiting requested review from Devconf Devconf is a code owner

@seongbin9786 seongbin9786 Awaiting requested review from seongbin9786 seongbin9786 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Student and Manager Created Event
2 participants
@enrouteeee @seongbin9786