Repository navigation
Feat/74 회원 탈퇴 구현 - #253
Open
jaylovegood wants to merge 18 commits into
Hidden character warning
The head ref may contain hidden characters: "feat/74-\ud68c\uc6d0-\ud0c8\ud1f4-\uad6c\ud604"
Open
Feat/74 회원 탈퇴 구현#253jaylovegood wants to merge 18 commits into
jaylovegood wants to merge 18 commits into
Conversation
# Conflicts: # src/main/kotlin/com/wafflestudio/spring2025/domain/user/controller/UserController.kt
Open
uykhc
approved these changes
Jun 6, 2026
uykhc
left a comment
Collaborator
There was a problem hiding this comment.
작업하느라 수고하셨습니다!
- 이벤트 시작 시점 및 신청기간 마감 여부에 따라 삭제/익명화 전략 다르게 구현
- eventIds.sorted()로 락 순서 보장
- AfterCommit 패턴으로 메일 발송 지연
- DB 마이그레이션 내 Nullable 처리
- 테스트 커버리지
회의 때 말씀하셨던 내용대로 구현하신 부분 모두 확인했습니다.
크리티컬한 부분은 없어보이지만 굳이 개선할 부분을 찾아보면
- UserService의 고아 데이터 처리 부분 로그 남기기
- EventService의 afterCommit 이메일 발송 실패 시 로그 남기기
- 같은 이벤트 참여자 동시 탈퇴 테스트 시나리오 추가
- UserService에서 DB 쿼리 조회가 2번 발생하는데 모든 참여 등록을 한 번에 조회하고 partition으로 분리하여 DB 조회 최적화
정도가 있는 것 같습니다.
감사합니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✨ Feature PR (to dev)
🧪 로컬 테스트 여부 (작업자 체크)
src/main/resources/static/docs/openapi.yaml을 함께 갱신했습니다./docs/swagger-ui.html에서 변경 API가 의도대로 보이는지 확인했습니다.📄 documentation 최신화 여부
📌 작업 내용(what & why)
회원 탈퇴 처리 및 이에 수반되는 event/registration 삭제 및 익명화 로직을 구현하였습니다.
핵심 도메인 로직 (UserService.deleteMe())
수정된 주요 파일
테스트
Swagger
🔎 영향 범위
📡 API 변경사항 (있다면)
🔥 관련 이슈
*이메일 비동기화 브랜치와 충돌이 있을 것으로 예상됩니다.