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.

Sign up for GitHub

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

Jump to bottom

회원 탈퇴 API 수정 #122

Merged
merged 5 commits into from
Apr 28, 2024
Merged

회원 탈퇴 API 수정 #122

merged 5 commits into from
Apr 28, 2024

Conversation

chaewss
Copy link
Member

@chaewss chaewss commented Apr 24, 2024

🎯 Issue

🔑 Key Changes

  • user_id의 Not Null 제약조건을 삭제했습니다.
  • 사용자 탈퇴 후 30일이 지나면 userId를 null로 초기화하는 스케줄러를 추가했습니다.
  • 탈퇴한 사용자가 로그인 한 경우 예외 처리를 NotFound에서 Forbidden으로 변경했습니다.

@chaewss chaewss requested a review from feel-coding April 24, 2024 13:20
@chaewss chaewss self-assigned this Apr 24, 2024
@github-actions GitHub Actions
Copy link

Test Results

81 tests   81 ✅  2s ⏱️
 8 suites   0 💤
 8 files     0 ❌

Results for commit 276a63c.

feel-coding
feel-coding approved these changes Apr 28, 2024
View reviewed changes
Copy link
Member

@feel-coding feel-coding left a comment

Choose a reason for hiding this comment

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

고생하셨습니다👍

@feel-coding feel-coding merged commit a4a6305 into develop Apr 28, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@feel-coding feel-coding feel-coding approved these changes

Assignees

@chaewss chaewss

Labels
refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chaewss @feel-coding