-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor: 알림 시스템 이벤트 기반 리팩토링 및 멀티플랫폼 지원 구현 완료 #615
Conversation
…별로 플랫폼을 지정하여 전송할 수 있도록 변경
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이번 작업으로 이벤트-구독 구조의 특징 및 장점을 잘 파악할 수 있었던 것 같습니다!
인터페이스 활용과 헥사고날 아키텍처의 특징에 맞게 알림 도메인을 리팩토링한 점이, 다른 알림 웹훅을 사용할 수 있는 확장성을 고려한 작업이라 앞으로 유지보수가 편해질 것 같습니다.
...pi/global/common/notificationSetting/adapter/in/web/NotificationSettingUpdateController.java
Outdated
Show resolved
Hide resolved
.../clab/api/global/common/notificationSetting/adapter/out/webhook/SlackNotificationSender.java
Show resolved
Hide resolved
...clab/api/global/common/notificationSetting/application/event/ApplicationStartupListener.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
작업량이 상당하셨을 것 같은데 고퀄리티로 작업해주셔서 감사합니다..! 코드의 질에 항상 놀라고 많이 배우고 있습니다
...java/page/clab/api/domain/auth/login/application/service/TwoFactorAuthenticationService.java
Show resolved
Hide resolved
src/main/java/page/clab/api/global/auth/filter/InvalidEndpointAccessFilter.java
Show resolved
Hide resolved
src/main/java/page/clab/api/global/auth/filter/InvalidEndpointAccessFilter.java
Show resolved
Hide resolved
...lab/api/global/common/notificationSetting/adapter/out/webhook/DiscordNotificationSender.java
Show resolved
Hide resolved
...pi/global/common/notificationSetting/adapter/in/web/NotificationSettingUpdateController.java
Outdated
Show resolved
Hide resolved
…nageNotificationSettingUseCase 변경
Summary
알림 시스템을 이벤트 기반 구조로 리팩토링하고 다중 플랫폼 지원을 구현했습니다. 헥사고날 아키텍처를 적용하여 도메인 로직과 인프라스트럭처 코드를 분리함으로써 시스템의 유지보수성과 확장성을 향상시켰습니다.
Tasks