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

[Feature] 모집자 회원 탈퇴 API 구현 #231

Merged
merged 3 commits into from
Jun 3, 2024

Conversation

hojeong2747
Copy link
Member

💡 연관된 이슈

close #223

📝 작업 내용

  • 모집자 회원 탈퇴 API 구현

💬 리뷰 요구 사항

봉사자와 마찬가지로 탈퇴한 계정 표시를 위한 모집자 계정을 하나 추가했습니다.
이때 구분을 위해 deletedVolunteer/Intermediary를 추가해 계정 정보를 넣어주었습니다.

Intermediary 엔티티와 연관관계에 있는 엔티티들은 다음과 같습니다.

  1. Post
  2. Application
  3. IntermediaryFcm
  4. IntermediaryNotification
  5. DogStatus

1,2 번은 데이터를 삭제하지 않고 '탈퇴한 사용자'임을 표시해 주기로 하였고 3,4번은 삭제가 필요합니다.

근황 엔티티는 현재 사용하지 않음에도 import.sql 실행되어 근황 데이터가 들어가서 삭제를 해주어야 탈퇴가 가능했습니다. 따라서 근황 관련 데이터 insert문을 주석처리했습니다!

추가로, Volunteer 탈퇴 시 VolunteerFcm, VolunteerNotification까지 벌크 연산으로 삭제 처리했습니다.

  • 탈퇴한 모집자의 프로필을 보게 될 때의 처리를 어떻게 할 것인지 정해야 해요. 리뷰, 공고, 신청 응답 API에서 intermediaryId 필드를 보고 탈퇴한 사용자의 id인 경우 1) 모집자 프로필 보기 버튼을 없애거나 2) 버튼은 그대로 두고, 눌렀을 경우 탈퇴한 모집자임을 명시해 주면 될 것 같습니다. 이건 프론트에게 선택권을 주도록 해볼게요!

추후 테스트 계정 여러 개 두고 탈퇴 후 여러 테스트 케이스로 후기, 공고, 신청 등의 데이터를 확인해 보면 좋을 것 같습니다!

@hojeong2747 hojeong2747 added ✨ Feature 기능 개발 Priority: Medium 우선순위 중간 🐰 Hojeong 담당자 labels Jun 3, 2024
@hojeong2747 hojeong2747 merged commit e707454 into develop Jun 3, 2024
1 check passed
@hojeong2747 hojeong2747 deleted the feat/223-delete-withdraw-api branch June 3, 2024 15:17
Copy link
Member

@kyeong-hyeok kyeong-hyeok left a comment

Choose a reason for hiding this comment

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

알림 목록과 FCM 토큰까지 삭제 대상에 추가한 것 좋습니다! 탈퇴한 사용자를 어떻게 보여줄지는 프론트와 협의해보면 좋을 거 같아요 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🐰 Hojeong 담당자 Priority: Medium 우선순위 중간
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 모집자 회원 탈퇴 API 구현
2 participants