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/user withdrawal] 마이페이지-회원탈퇴 API #67

Merged
merged 5 commits into from
Nov 26, 2023

Conversation

dldmsql
Copy link
Member

@dldmsql dldmsql commented Nov 25, 2023

작업 내용

마이페이지-회원탈퇴 API 개발

관련 이슈

#66

작업 확인 방법

swagger-ui를 통해 확인 가능

추가 정보 (선택 사항)

  • 회원 탈퇴 테이블 PK 값을 User 테이블 PK로 지정하여 식별 관계로 구성했습니다. OneToOne이기도 하고, 탈퇴에 대한 정책이 명확하지 않아 1명의 유저가 1번의 탈퇴 기록을 갖는다.로 정의하고 구성했습니다.

  • 회원 탈퇴 사유를 '기타'로 선택할 경우, 추가 사유를 작성하는 UI가 있습니다. 이 부분에서 글자수 제한 정책이 명확하지 않아 최대 100글자로만 정의하였습니다. 즉, 아무 글자를 입력하지 않아도 저장이 되도록 구현되어 있습니다.

🙋🏻‍♀️ 질문

  1. 회원이 탈퇴를 할 경우, 앱에서 팝업으로만 탈퇴를 명시하나요?

이 방법 외에 생각한 것은 이메일 주소를 알고 있고, 논리 삭제 방식이니 탈퇴 이후 사용자 이메일 주소로 탈퇴 사실을 한번더 통지하는 것을 생각했습니다.

  1. 논리 삭제 방식으로 탈퇴 상태를 구분하고 있는데, 탈퇴 유저가 로그인이나 회원가입을 시도했을 때의 정책은 어떻게 되는 건가요?

신규 유저로 볼 것인지 기존 데이터를 다시 제공할 것인지가 궁금합니다.

@dldmsql dldmsql added the Feature 기능 개발 label Nov 25, 2023
@dldmsql dldmsql requested a review from Qbeom0925 November 25, 2023 15:41
@dldmsql dldmsql self-assigned this Nov 25, 2023
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

80.0% 80.0% Coverage
0.0% 0.0% Duplication

@dldmsql
Copy link
Member Author

dldmsql commented Nov 26, 2023

@Qbeom0925 디자인팀에서 탈퇴 사유 글자수 제한이 없다고 해서 text 타입으로 변경해두었습니다.

Comment on lines +10 to +14
NOT_USE_USUALLY("앱을 잘 쓰지 않아요"),
INCONVENIENT_IN_TERMS_OF_USABILITY("사용성이 불편해요"),
ERRORS_OCCUR_FREQUENTLY("오류가 자주 발생해요"),
MY_SCHOOL_HAS_CHANGED("학교가 바뀌었어요"),
ETC("기타");
Copy link
Member

Choose a reason for hiding this comment

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

Enum으로 탈퇴사유 정리해두는 것 좋은 것 같습니다!!

Comment on lines +21 to +22
@Schema(description = "사유가 '기타'일 경우, 추가 이유 입력해주세요.", example = "다른 서비스를 사용하게 되었다.")
String etcReason) {}
Copy link
Member

Choose a reason for hiding this comment

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

만약 기타를 선택한 경우에 대해서도 남겨서 서비스 운영도 생각하는 것이 너무 좋은 방향 같아요!

@Qbeom0925 Qbeom0925 merged commit 666c7dc into develop Nov 26, 2023
@Qbeom0925
Copy link
Member

회원이 탈퇴를 할 경우, 앱에서 팝업으로만 탈퇴를 명시하나요?
이 방법 외에 생각한 것은 이메일 주소를 알고 있고, 논리 삭제 방식이니 탈퇴 이후 사용자 이메일 주소로 탈퇴 사실을 한번더 통지하는 것을 생각했습니다.

논리 삭제 방식으로 탈퇴 상태를 구분하고 있는데, 탈퇴 유저가 로그인이나 회원가입을 시도했을 때의 정책은 어떻게 되는 건가요?
신규 유저로 볼 것인지 기존 데이터를 다시 제공할 것인지가 궁금합니다.

해당 질문은 이번주 회의에 논의를 해보는 것도 좋은 것 같습니다!!

일단 피알은 반영 해두도록 하겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants