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.

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

[MODIFY] 유저 탈퇴시 이벤트 기반으로 삭제해야하는 데이터 삭제 처리 로직 추가 #426

Merged
merged 25 commits into from
Nov 5, 2024
Merged

[MODIFY] 유저 탈퇴시 이벤트 기반으로 삭제해야하는 데이터 삭제 처리 로직 추가 #426

merged 25 commits into from
Nov 5, 2024

Conversation

kseysh
Copy link
Member

@kseysh kseysh commented Oct 31, 2024

📝 PR Summary

유저 탈퇴시 삭제해야 하는 데이터를 삭제하는 로직을 추가하였습니다.
데이터 삭제를 이벤트 기반으로 하여 강한 의존성 결합을 최소화하였습니다.
기존 전역 변수로 관리되던 EventPublisher 객체를 컴포넌트로 관리하도록 하여 테스트에 용이하도록 하였습니다.

🌴 Works

  • 유저 탈퇴 시 아래 항목들을 삭제하도록 변경
  • UserFortune
  • PokeHistory
  • SoptampUser
  • Stamp
  • Friend
  • Notification
  • PushToken

🌱 Related Issue

closed #336

🌵 PR 참고사항

  • Event 테스트 방안을 고안해봤는데 일단 SpringBootTest로 구현하여 테스트 속도가 조금 느려졌습니다..!

kseysh added 25 commits October 30, 2024 21:06
@kseysh
[refactor] SoptampFacade에 Transactional 추 (#336)
a16c1e6
@kseysh
[refactor] 사용하지 않는 메서드 삭제 (#336)
bfaee56
@kseysh
[refactor] 솝탬프 유저 삭제 메서드 추 (#336)
0265998
@kseysh
[refactor] 알림 삭제 메서드 추가 (#336)
70f6fec
@kseysh
[refactor] 불필요한 User 객체 전달을 userId 전달로 변경 (#336)
3b503aa
@kseysh
[refactor] 불필요한 EventHandler 삭제 처리 (#336)
1aed275
@kseysh
[refactor] 유저 탈퇴시 유저 삭제 Event 추 (#336)
2afbba0
@kseysh
[refactor] 유저 탈퇴시 솝탬프 유저 삭제 Event 추가 (#336)
5b8b1c1
@kseysh
[refactor] 스탬프 삭제시 스탬프 s3 이미 삭제 Event 추가 (#336)
6108774
@kseysh
[refactor] 유저 탈퇴시 pokeHistory 삭제 Event 추가 (#336)
100e51e
@kseysh
[refactor] 유저 탈퇴시 Friend 삭제 Event 추가 (#336)
2d6ca83
@kseysh
[refactor] 유저 탈퇴시 Notification 삭제 Event 추가 (#336)
0cb76c8
@kseysh
[refactor] 유저 탈퇴시 PushToken 삭제 메서드 실행 (#336)
bef060c
@kseysh
[refactor] 유저 탈퇴시 유저 서비스에서 이벤트 기반으로 탈퇴 로직을 처리하도록 변경 (#336)
5290875
@kseysh
[refactor] 유저 탈퇴시 관련한 오늘의 운세 데이터 삭제 (#336)
76f66c0
@kseysh
[refactor] 유저 탈퇴시 파일 삭제 이벤트 발생 테스트 추 (#336)
b468ad3
@kseysh
[refactor] 가독성있는 람다 표현식으로 변 (#336)
428fe9d
@kseysh
[refactor] 유저 탈퇴 책임을 별도의 객체로 관 (#336)
534ab36
@kseysh
[refactor] static이 아닌 싱글톤을 이용해 EventPublisher 관리하도록 변경 (#336)
62d39c7
@kseysh
[refactor] static이 아닌 싱글톤을 이용해 EventPublisher 관리하도록 변경 (#336)
1dc32ca
@kseysh
[refactor] 테스트용 기본 yml 변경 (#336)
d114716
@kseysh
[refactor] EventPublisherTest 추가 (#336)
edc486b
@kseysh
[refactor] 찌르기 실행시 PokeEvent 발생 이벤트 검증 테스트 추가 (#336)
a4e71ff
@kseysh
[feat] 스탬프 삭제시 PokeEvent 발생 스탬프 삭제 및 s3이미지 삭제 검증 테스트 추가 (#336)
994c024
@kseysh
[refactor] 유저 삭제 테스트 책임 이관 (#336)
a0f3498
@kseysh kseysh added 🔨 Refactor 코드 개선 🔧 Modify 기능 수정 labels Oct 31, 2024
@kseysh kseysh requested a review from rlarlgnszx October 31, 2024 07:15
@kseysh kseysh self-assigned this Oct 31, 2024
@height Height
Copy link

height bot commented Oct 31, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

rlarlgnszx
rlarlgnszx approved these changes Nov 5, 2024
View reviewed changes
Copy link
Contributor

@rlarlgnszx rlarlgnszx left a comment

Choose a reason for hiding this comment

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

오! Event 개념을 단순히 비동기 처리로직으로만 구현해서 잘못 알고계신분들이 많은데
제대로 Event로직을 작성하신것 같아요! 💯
수고하셨습니다.

@kseysh kseysh merged commit 14089b6 into dev Nov 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rlarlgnszx rlarlgnszx rlarlgnszx approved these changes

Assignees

@kseysh kseysh

Labels
🔧 Modify 기능 수정 🔨 Refactor 코드 개선 size/XL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[MODIFY] 유저 탈퇴 시 삭제되어야 하는 데이터들 모두 삭제하도록 수정
2 participants
@kseysh @rlarlgnszx