Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: 회원탈퇴 기능 추가 #223

Merged
merged 13 commits into from
Jun 20, 2024
Merged

Conversation

prgmr99
Copy link
Member

@prgmr99 prgmr99 commented Jun 20, 2024

구현 내용/방법

간단하게 구현한 내용과 방법에 대한 설명

  • 회원탈퇴 기능을 구현했습니다.
  • 회원탈퇴 시, 헤더를 제거하여 사용자가 중간에 이탈하는 경우를 방지했습니다.
  • 계정관리 UI를 임시로 일부 변경했습니다. (회원탈퇴만 존재하도록)
  • 회원탈퇴 모달을 띄워 최종 확인을 진행합니다.
  • 회원탈퇴를 클릭할 시, 모든 관련 캐시를 제거하고, 토큰, platform id, userState을 제거했습니다.
  • 회원탈퇴 이후 바로 signin 페이지로 이동하는 것을 수정하기 위해 PrivateRouter를 일부 수정했습니다. (onSuccess navigate('/') 이 먹히지 않음.)

To Do

  • 회원탈퇴 UI 일부 수정
  • 회원탈퇴 API 연결
  • 로그인 상태 업데이트 로직 추가
  • 회원가입 페이지에서 헤더 제거

리뷰 필요

나중에 다시 고민해야할 내용이 있는 내용

없을 경우 작성 X

있을 경우 작성 후 이슈 남기고 해당 PR 링크

  • 궁금한 점이 있다면 말씀해주세요!

close #207

@prgmr99 prgmr99 added Priority: High 우선순위: 높음 Status: Review Needed 상태: 리뷰 대기중인 상태 Type: Feature 유형: 기능 추가 Type: Improvement 유형: 기능 개선 labels Jun 20, 2024
@prgmr99 prgmr99 self-assigned this Jun 20, 2024
@prgmr99 prgmr99 changed the title feat: 회원가입 기능 추가 feat: 회원탈퇴 기능 추가 Jun 20, 2024
return (
<LocalizationProvider dateAdapter={AdapterDateFns} adapterLocale={ko}>
<QueryClientProvider client={queryClient}>
<RecoilRoot>
<Header />
{!location.pathname.includes('#') && <Header />}
Copy link
Contributor

Choose a reason for hiding this comment

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

오 헤더 제거해주셨군요!!👍🏻👍🏻

Copy link
Contributor

Choose a reason for hiding this comment

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

모달 폴더 위치를 recruit에 넣어놓은 이유가 궁금합니다!

Copy link
Member Author

Choose a reason for hiding this comment

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

프로젝트 초기에 잘못 설정한 것을 아직 못 바꾼거 같습니다;;
리팩토링때 수정해보겠습니다!!

@prgmr99 prgmr99 merged commit 4b018d0 into release-1.0 Jun 20, 2024
@kimsuyeon0916 kimsuyeon0916 deleted the feature/#207_account_secession branch November 6, 2024 04:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Priority: High 우선순위: 높음 Status: Review Needed 상태: 리뷰 대기중인 상태 Type: Feature 유형: 기능 추가 Type: Improvement 유형: 기능 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 회원탈퇴 기능
2 participants