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

[REFACTOR]/#88- Member 엔티티의 필드를 변경하여 id와 userName으로 관리하기 #90

Merged
merged 5 commits into from
Apr 5, 2024

Conversation

koojun99
Copy link
Contributor

@koojun99 koojun99 commented Apr 5, 2024

🚀 개요

기존의 String 타입의 id로 Member를 관리하던 방식에서(인스타그램 느낌)
DB에 저장할 Long 타입의 고유 id 값 + String 타입의 고유 userName값으로 분리하는 작업을 진행했습니다.

🔍 변경사항

  • checkId 대신 checkName로 메서드명 변경
  • Member 엔티티 - id + userName 필드 구조로 변경
  • 다른 비즈니스 로직에서 Member의 id를 사용하는 부분 Long 타입으로 형변환

⏳ 작업 내용

  • checkId 대신 checkName로 메서드명 변경
  • Member 엔티티 - id + userName 필드 구조로 변경
  • 다른 비즈니스 로직에서 Member의 id를 사용하는 부분 Long 타입으로 형변환

📝 논의사항

Spring Security와 JWT 토큰 관련해서 memberId가 Long 타입으로 바뀐 것이 문제를 일으키진 않을지 궁금합니다!
(우선은 createAccessToken이나 toMemberSecurity에는 String.valueOf와 Long.valueOf을 이용해 넘겨주도록 수정했습니다.)

@koojun99 koojun99 added the ♻️ refactor Refactoring code label Apr 5, 2024
@koojun99 koojun99 requested a review from CYY1007 April 5, 2024 03:34
@koojun99 koojun99 self-assigned this Apr 5, 2024
@koojun99 koojun99 linked an issue Apr 5, 2024 that may be closed by this pull request
1 task
@koojun99 koojun99 changed the base branch from develop to refactor/87 April 5, 2024 03:36
@koojun99 koojun99 changed the base branch from refactor/87 to develop April 5, 2024 03:37
@CYY1007 CYY1007 merged commit 7657ea0 into develop Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor Refactoring code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🔨[refactor] userId와 userName의 분리
2 participants