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

탈퇴시 회원정보 삭제 #78

Merged
merged 2 commits into from
Feb 26, 2024
Merged

탈퇴시 회원정보 삭제 #78

merged 2 commits into from
Feb 26, 2024

Conversation

JaeyoungChoi98
Copy link
Contributor

이슈 연결

구현한 API

  • x

작업 사항

회원 탈퇴시 회원정보 삭제
그로 인해 activeStatus.EXIT 상태 삭제

기획시 이용약관에 탈퇴시 회원정보를 바로 삭제해야하기 때문에 바로 삭제 (단, 특정상황시에는 저장(부정고객 등))

리뷰 요청 사항

@JaeyoungChoi98 JaeyoungChoi98 added the Refactor 코드 리팩토링 label Feb 26, 2024
@JaeyoungChoi98 JaeyoungChoi98 self-assigned this Feb 26, 2024
@@ -61,7 +61,7 @@ public class Member extends BaseTimeEntity {
@OneToMany(mappedBy = "member", fetch = FetchType.LAZY, orphanRemoval = true)
private final List<WantClubCategory> wantClubCategories = new ArrayList<>();

@OneToMany(mappedBy = "reportedMember", fetch = FetchType.LAZY)
@OneToMany(mappedBy = "reportedMember", fetch = FetchType.LAZY, orphanRemoval = true)
Copy link
Member

Choose a reason for hiding this comment

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

탈퇴한 멤버를 바로 삭제하지만, 신고당한 멤버에 대한 정보는 저장해야 한다면 orphanRemoval = true는 걸면 안될꺼 같아요.
그래서 2가지 방법으로 처리할 수 있을 것 같아요

  1. 실제로는 삭제 안함 -> 일정 기간뒤 삭제
  2. 실제로 바로 삭제하되, 외래키로 된 관계 끊기

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 머야 신고멤버였구나 수정하겠습니다

@JaeyoungChoi98 JaeyoungChoi98 merged commit ec02636 into dev Feb 26, 2024
1 check passed
@JaeyoungChoi98 JaeyoungChoi98 deleted the Feat/#77/quitMember branch February 26, 2024 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants