Skip to content

Conversation

@dvlp-sy
Copy link
Collaborator

@dvlp-sy dvlp-sy commented Mar 18, 2025

#️⃣ 연관된 이슈

ex) #이슈 번호, #이슈 링크


🔎 작업 내용

기능에서 어떤 부분이 구현되었는지 설명해주세요.

  • 회원탈퇴 API 구현 -> inactive 필드 사용 (회원탈퇴 후 특정 기간이 지난 후 DB에서 자동으로 삭제하도록 구현)
  • 로그인 및 회원가입 시 inactive 필드 확인하도록 변경
    • inactive != null : 기존의 회원 정보를 삭제하고 새로 회원가입 가능
    • inactive == null : 이메일이 이미 존재하는 경우 새로 회원가입 불가

📷 스크린샷 (선택)

작업한 결과물에 대한 간단한 스크린샷을 올려주세요.


💬리뷰 요구사항 (선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

로그인/회원가입 기능 변경사항에 대해 테스트를 진행하기는 했으나 오류가 있을 수 있습니다.
번거롭더라도 같이 확인해주시면 감사하겠습니다 😃

@dvlp-sy dvlp-sy added the 👍 feature New feature or request label Mar 18, 2025
@dvlp-sy dvlp-sy requested review from FromKyoung and msk226 March 18, 2025 10:06
@dvlp-sy dvlp-sy self-assigned this Mar 18, 2025
@dvlp-sy dvlp-sy linked an issue Mar 18, 2025 that may be closed by this pull request
Copy link
Member

@msk226 msk226 left a comment

Choose a reason for hiding this comment

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

오 역시 굉장히 빠르고 정확하게 만들어주셨군요... 수고하셨습니다!!!

그런데 문득 든 생각인데 임시 탈퇴한 회원이 다시 복귀 하기 위한 API도 필요할까요???

@dvlp-sy
Copy link
Collaborator Author

dvlp-sy commented Mar 18, 2025

오 역시 굉장히 빠르고 정확하게 만들어주셨군요... 수고하셨습니다!!!

그런데 문득 든 생각인데 임시 탈퇴한 회원이 다시 복귀 하기 위한 API도 필요할까요???

현재 화면설계서 상에서는 회원정보, 스터디정보를 즉시 삭제한다고 되어있어서 복구까지는 고려하지 않아도 될 것 같습니다!

그럼에도 inactive로 회원정보를 남겨둔 이유는 정말 호옥시나 나중에 탈퇴한 회원 정보가 필요한 상황이 생길까봐 그런 겁니닷 ㅎㅎ (신고당한 회원의 정보가 필요하다거나 법적인 문제에 연루될 가능성 등등이욥..!)

refreshTokenRepository.deleteAllByMemberIdIn(deletedMemberIds);

// 회원 정보 정리
memberRepository.deleteAllByIdInBatch(deletedMemberIds);
Copy link
Member

Choose a reason for hiding this comment

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

요걸로 다 지워버리면 관련 매핑 테이블들도 싹 다 지워지나요??? 저는 뭔가 막히는 것 같긴합니다

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

앗 우선은 member 엔티티에 orphanRemoval 설정이 되어 있어서 테스트했을 때는 정상적으로 지워졌습니다. 혹시 병합하고 문제 생기면 수동으로 삭제하는 방식으로 변경해야 할 것 같아요!

Copy link
Member

@msk226 msk226 Mar 18, 2025

Choose a reason for hiding this comment

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

아 데이터들이 예에에에에전에 넣어놓은 데이터라 orphanRemoval이 적용 안된 것 같네요!!!

확인했습니다!!!! 고생하셨어요! 머지 하셔도 될 것 같습니당

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵! 꼼꼼하게 봐주셔서 감사합니다 마커스님 😄

@dvlp-sy dvlp-sy merged commit 8a3f7e3 into develop Mar 18, 2025
1 check passed
@msk226 msk226 deleted the SPOT-234/feat branch March 31, 2025 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👍 feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 회원탈퇴 기능 구현

3 participants