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

[#45, #46, #56] 유저 이메일 찾기, 비밀번호 찾기, 회원 복구 #69

Merged
merged 13 commits into from
Aug 16, 2022

Conversation

memoer
Copy link
Contributor

@memoer memoer commented Aug 10, 2022

Related Issues

Description

이메일 찾기

  • "이름, 생년월일" 을 통해 유저 아이디 찾기를 진행합니다.
  • "생년월일"이 아이디 찾기에서 사용되기 때문에, 회원가입시 "생년월일" 데이터를 필수적으로 넘겨야 하는 식으로 변경하였습니다.

비밀번호 찾기

  • 비밀번호 변경시 생성된 코드는 session으로 사용하는 redis에 저장하고, 이를 3분간 유지합니다.
  • resetPassword 를 통해 비밀번호 변경 완료시, session으로 사용하는 redis에서 해당 code를 제거합니다.
  • 휴대폰을 통한 비밀번호 변경은 잠시 보류했습니다.

회원복구

  • 로그인 이후, 해당 유저가 탈퇴한 유저라면 복구 API 를 사용할 수 있습니다. [PUT - /users/me?type=restore]
  • 회원 복구 생각한 흐름 [Flow]
    1. AuthService.logIn 에서 반환하는 유저 데이터는 회원 탈퇴한 유저까지 포함해서 반환합니다.
    2. 로그인 이후, 회원 탈퇴했던 유저라면 회원 복구 API를 통해 복구할 수 있습니다.

변경 사항

변경 사항들을 적어주세요

질문 사항

질문 사항들을 적어주세요

기타

그 외로 적고 싶은 내용을 적어주세요

(Optaionl) 어떻게 테스트하셨나요?

@memoer memoer added Type: Feature 기능 개발 Priority: Second 2번째로 처리되야 하는 이슈입니다. labels Aug 10, 2022
@memoer memoer added this to In progress in AirJnC Project via automation Aug 10, 2022
@memoer memoer added this to the [UseCases] 사용자 - 인증 milestone Aug 10, 2022
@memoer memoer added Priority: Third 3번째로 처리되야 하는 이슈입니다. and removed Priority: Second 2번째로 처리되야 하는 이슈입니다. labels Aug 10, 2022
@memoer memoer force-pushed the feat/#45,#46,#56 branch 3 times, most recently from 7ac75cd to 3873ea3 Compare August 11, 2022 13:13
@memoer memoer changed the title [#45, #46, #56] 유저 이메일 찾기, 비밀번호 찾기, 회원 복구 (WIP) [#45, #46, #56] 유저 이메일 찾기, 비밀번호 찾기, 회원 복구 Aug 12, 2022
@memoer memoer changed the title (WIP) [#45, #46, #56] 유저 이메일 찾기, 비밀번호 찾기, 회원 복구 [#45, #46, #56] 유저 이메일 찾기, 비밀번호 찾기, 회원 복구 Aug 12, 2022
@f-lab-michael
Copy link
Collaborator

이 PR도 이 PR에 필요한 내용만 남기도록 해주세요

  • 기존 머지된 pr삭제

별도 pr로 분리하는경우에 디펜던시가 없는경우 미리 정리해놓으면 리뷰과정이 더 빠르게 진행될수있습니다.

@memoer memoer merged commit 10a4be5 into master Aug 16, 2022
AirJnC Project automation moved this from In progress to Done Aug 16, 2022
@memoer memoer deleted the feat/#45,#46,#56 branch September 15, 2022 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Third 3번째로 처리되야 하는 이슈입니다. Type: Feature 기능 개발
Projects
Development

Successfully merging this pull request may close these issues.

[인증] 회원 복구 [인증] 유저 비밀번호 찾기 [인증] 유저 아이디 찾기
2 participants