Skip to content

feat: 모임 등록(Register) 이벤트에 대한 PaymentEventListener 구현#61

Merged
ZeroColaa merged 2 commits into
dev-v2from
feature/payment-v2
Jul 30, 2025
Merged

feat: 모임 등록(Register) 이벤트에 대한 PaymentEventListener 구현#61
ZeroColaa merged 2 commits into
dev-v2from
feature/payment-v2

Conversation

@ZeroColaa

@ZeroColaa ZeroColaa commented Jul 30, 2025

Copy link
Copy Markdown
Member
  • PaymentEventListener에 @eventlistener 추가하여 Register 이벤트 수신

  • 메세지 허브에서 알림 메세지는 필요한 경우에만 발행되도록 meeting쪽 처리

- Register 이벤트 수신 시 결제 로직 수행
- 무료/유료 모임 분기 처리
- 결제 완료 시 PaymentCompletedEvent 발행

@Zyooon Zyooon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다.

@hojin915 hojin915 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다


import java.util.List;

public class MeetingMessageEvents {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

조금 헷갈려서 그런데 MeetingEvents는 그대로 있고 MeetingMessageEvents는 따로 있는건가요?

@hojin915 hojin915 Jul 30, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

에러가 발생할 수 있어서 MeetingEvents를 일단 둔 게 아닐까 싶습니다
MeetigEvents를 사용하던 이벤트들은 MeetingMessageEvents로 넘어가야합니다

정확히는 제가 따로 설정한 이벤트 때문에 MeetingEvents 내부에 구분이 필요해져서 분리했고
제가 따로 만든 Events만 따로 분리하고 클래스명을 명시적으로 바꾼 상태입니다

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아 넘어가야되는군요 알겠습니다

@ZeroColaa ZeroColaa merged commit 0a5d5d7 into dev-v2 Jul 30, 2025
1 check passed
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.

4 participants