Skip to content

Feature 69 add student or manager created event #76

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
Open
enrouteeee wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Feature 69 add student or manager created event #76

enrouteeee wants to merge 3 commits into from

Conversation

@enrouteeee
Copy link
Member

@enrouteeee enrouteeee commented Nov 25, 2021

close #69

@seongbin9786
Copy link
Member

seongbin9786 commented Nov 25, 2021

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

@seongbin9786
Copy link
Member

seongbin9786 commented Nov 25, 2021

도메인 이벤트를 발행하는 것은 애그리거트, 엔터티의 역할이고, 트랜잭션에 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

3 participants

@enrouteeee @seongbin9786