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

feat: 랜덤 닉네임 기능 도입에 따른 GUEST 상태 제거 #245

Merged
merged 10 commits into from
Jan 28, 2024

Conversation

uwoobeat
Copy link
Member

🌱 관련 이슈

📌 작업 내용 및 특이사항

  • MemberRole 에서 GUEST 상태를 제거했습니다.
  • 닉네임 변경 및 GUEST -> USER 상태로 업데이트하는 register() 관련 메서드를 제거했습니다.
  • ID/PW 관련 메서드를 삭제했습니다. (createGuestMember(String username, String password))
  • ID/PW 관련 컨트롤러 / 서비스 메서드를 deprecated 처리했습니다.
    • 게스트 관련 로직의 경우 아무런 동작을 하지 않도록 수정했습니다 (// do nothing 주석 참고)
  • 소셜 로그인 response의 isGuest 필드는 항상 false를 리턴합니다.

📝 참고사항

  • 현재 테스트 시 createNormalMember(Profile profile) 을 사용하고 있는데, 테스트 전용 생성자를 사용하는 것은 지양해야 하므로 createNormalMember(OauthInfo info, String nickname) 으로 대체해야 합니다.
  • 이에 따라 createNormalMember(Profile profile) 을 deprecated 처리했습니다.

📚 기타

@uwoobeat uwoobeat requested review from kdomo and char-yb January 28, 2024 16:32
@uwoobeat uwoobeat changed the title feat: 랜덤 닉네임 기능 도입에 따른 GUEST 상태 제거 feat: 랜덤 닉네임 기능 도입에 따른 GUEST 상태 제거 Jan 28, 2024
Copy link
Member

@kdomo kdomo left a comment

Choose a reason for hiding this comment

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

deprecate처리 확인 했습니당

Copy link

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

68 New issues
0 Security Hotspots
52.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@uwoobeat uwoobeat merged commit 917d1f7 into develop Jan 28, 2024
2 checks passed
@uwoobeat uwoobeat deleted the feature/230-remove-guest branch January 28, 2024 16:44
@github-actions github-actions bot added the merged 머지된 PR label Jan 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged 머지된 PR
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

✨ 랜덤 닉네임 기능 도입에 따른 GUEST 상태 제거
2 participants