Skip to content

#24 회원 탈퇴#53

Merged
yunjaeeun merged 1 commit intodevelopfrom
51-24-회원-탈퇴
Dec 18, 2024

Hidden character warning

The head ref may contain hidden characters: "51-24-\ud68c\uc6d0-\ud0c8\ud1f4"
Merged

#24 회원 탈퇴#53
yunjaeeun merged 1 commit intodevelopfrom
51-24-회원-탈퇴

Conversation

@kingkang85
Copy link
Copy Markdown
Contributor

📋 요약

  • 회원 탈퇴 시 회원 정보를 삭제합니다.

🛠 변경 사항

  • 이번 Pull Request에서 작업한 주요 변경 사항은 다음과 같습니다:
    • 새로운 기능 추가 또는 버그 수정
    • 코드 리팩토링 또는 최적화
    • 문서 업데이트

🔗 관련 이슈

📸 스크린샷 또는 GIF (해당되는 경우)

  • 기존 tbl_user
    image

  • 6번 유저 로그인 후 탈퇴
    image

  • 탈퇴 후 tbl_user
    image

✅ 체크리스트

  • 코드가 정상적으로 동작하는지 테스트했습니다.
  • 관련 문서를 작성하거나 업데이트했습니다. (해당되는 경우)
  • 변경 사항을 반영한 테스트 코드를 추가했습니다.
  • 모든 테스트가 성공적으로 통과했습니다.

🛡 테스트 방법

.

📚 추가 참고 사항

일단 회원 탈퇴 하면 정보를 삭제하는 것으로 구현하였는데, 게시물이나 댓글 같은 경우는 함께 삭제를 할 지, 익명화 처리를 하여 남겨둘 지 이야기 해보면 좋을 것 같습니다..!

- 회원 탈퇴 시 회원 정보 삭제
- 회원 탈퇴 시 로그아웃을 통해 토큰 블랙리스트 등록

Related to: #51
Copy link
Copy Markdown
Contributor

@MinFe4990 MinFe4990 left a comment

Choose a reason for hiding this comment

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

!check

Copy link
Copy Markdown
Contributor

@LeeGukgeon LeeGukgeon left a comment

Choose a reason for hiding this comment

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

회원탈퇴 기능 확인했습니다~

Copy link
Copy Markdown
Contributor

@uni-zun uni-zun left a comment

Choose a reason for hiding this comment

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

check!!!

Copy link
Copy Markdown
Contributor

@KIM9909 KIM9909 left a comment

Choose a reason for hiding this comment

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

확인 했습니다 굿굿

@yunjaeeun
Copy link
Copy Markdown
Contributor

회원 정보 같은 경우 탈퇴일을 업데이트 하는 식으로 구현해도 괜찮을 것 같네요

@yunjaeeun yunjaeeun merged commit 084ccf6 into develop Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants