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]: GroupMember Payload, Event 및 FeignClient 수정 #28

Merged
merged 4 commits into from
Dec 10, 2023

Conversation

jihwan2da
Copy link
Member

@jihwan2da jihwan2da commented Dec 10, 2023

resolved #26
resolved #27

문제 상황

현재 GroupMember에 대한 이벤트는 groupMemberId(그룹 멤버 자체의 식별자)만 전달을 하고 있습니다.
근데 기존 서버에서는 GroupMember가 Group을 떠날 시 soft-delete가 아닌 그냥 삭제를 해버리기 때문에 groupMemberId를 통해서 FeignClient로 기존서버에 정보를 들고 올 때 문제가 발생합니다.

그에 따른 작업 고려 사항

  1. 기존 서버에 GroupMember 삭제를 soft-delete로 바꾼다.(고려한 해결방법)
    위의 방법으로 문제를 해결하는 것을 고려하였지만, 기존 서버의 수많은 로직들이 GroupMember와 엮여져 있어 변경사항이 전파되는 문제 때문에 위 방법으로는 해결하지 않았습니다.
  2. GroupMember 이벤트 Payload를 변경한다. (선택한 해결방법)
    GroupMember Payload를 groupMemberId가 아닌 userId, groupId로 변경하였습니다. groupMemberId 보다 user 및 group의 정보를 들고오기에 더 유연하다고 판단하였고, 이 변경으로 변경 전파가 크지 않다고 판단하였습니다.
  • 기존 GroupMemberEventPayload
data class GroupMemberPayload(
    val groupMemberId: Long,
    val eventType: GroupMemberPayloadType,
    val eventDateTime: LocalDateTime
)
  • 변경된 GroupMemberEventPayload
data class GroupMemberPayload(
    val userId: Long,
    val groupId: Long,
    val eventType: GroupMemberPayloadType,
    val eventDateTime: LocalDateTime
)

@jihwan2da jihwan2da added the fix label Dec 10, 2023
@jihwan2da jihwan2da self-assigned this Dec 10, 2023
@jihwan2da jihwan2da merged commit 348f777 into dev Dec 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant