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

소셜 로그인한 회원 탈퇴 기능 구현 #582

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

Mingyum-Kim
Copy link
Contributor

PR의 목적이 무엇인가요?

이슈 ID는 무엇인가요?

설명

회원 탈퇴는 회원 정보를 테이블에서 제거하는 것으로 구현합니다.
AccessToken이 있더라도 회원 테이블에서 조회가 불가능하다면 다시 로그인할 수 없습니다.

질문 혹은 공유 사항 (Optional)

만약 회원 탈퇴 후 재로그인한다면 현재 구현으로는 이메일 수집 동의 처럼 체크하는 화면이 뜨지 않을 것입니다.
이를 해결하기 위해서는 애플 서버에 토큰을 무효화하는 API를 보내야하는데, 우선은 우리 서버 단에서만 회원 탈퇴 시키는 것으로 구현하겠습니다.

@Mingyum-Kim Mingyum-Kim added BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) labels Sep 26, 2024
@Mingyum-Kim Mingyum-Kim self-assigned this Sep 26, 2024
Copy link
Contributor

@ay-eonii ay-eonii left a comment

Choose a reason for hiding this comment

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

토큰 무효화를 하지 않아도 앱 승인이 가능하다면
이대로 진행해도 괜찮을 것 같아요~👍

@@ -31,4 +32,10 @@ public ResponseEntity<RestResponse<MemberFindResponse>> findMyInfo(

return ResponseEntity.ok().body(new RestResponse<>(memberFindResponse));
}

@DeleteMapping("/delete")
public ResponseEntity<Void> removeMember(@LoginDarakbangMember DarakbangMember member) {
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분 나중에 헷갈리더라구요 ㅠㅠ

Suggested change
public ResponseEntity<Void> removeMember(@LoginDarakbangMember DarakbangMember member) {
public ResponseEntity<Void> removeMember(@LoginDarakbangMember DarakbangMember darakbangMember) {

Copy link
Contributor

@pricelees pricelees left a comment

Choose a reason for hiding this comment

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

고생했습니다~ 추후에 물리적 삭제가 아니라 논리적 삭제를 고려하는 등의 논의가 한번 있었으면 좋겠습니다!

@Mingyum-Kim Mingyum-Kim merged commit 9abfa4f into develop-backend Oct 10, 2024
1 check passed
@ss0526100 ss0526100 deleted the feature/#570 branch October 17, 2024 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants