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

fix: 그룹 관련 이벤트 핸들러 제거 및 리팩토링 #63

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

Coalery
Copy link
Member

@Coalery Coalery commented Sep 17, 2024

주요 변경 사항

그룹 관련 이벤트 및 이벤트 핸들러를 제거합니다.
또, 이에 의해 관련된 모든 클래스를 수정 및 리팩토링합니다.

  • 주로 이벤트 핸들러 내에서 처리하던 부분들을 각 커맨드 핸들러로 옮기고, 이에 따라 테스트 코드를 수정하는 부분이 주가 됩니다.

변경 이유

이벤트 개념을 운영하기엔 아래 이유 때문에 운영 비용이 상승합니다. 이에, 이벤트 대신 각 커맨드 핸들러에서 처리하는 방향으로 수정됩니다.

  • 규모가 작음
  • 비동기 핸들링이 까다로움
  • 트랜잭션을 따로 관리해주어야 함

@Coalery Coalery self-assigned this Sep 17, 2024
@Coalery Coalery merged commit 753f3ce into main Sep 17, 2024
3 checks passed
@Coalery Coalery deleted the fix/remove-group-event-handler branch September 17, 2024 18:19
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