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] Member 에서 email 필드 제거 #60

Open
3 tasks
JunRain2 opened this issue Apr 1, 2024 · 0 comments
Open
3 tasks

[Refactor] Member 에서 email 필드 제거 #60

JunRain2 opened this issue Apr 1, 2024 · 0 comments
Assignees
Labels
backend backend

Comments

@JunRain2
Copy link
Collaborator

JunRain2 commented Apr 1, 2024

Description

  • 자체 로그인을 구현하지 않기 때문에, 필요 없어진 email 필드를 제거하는 작업
  • 추후, 멤버 엔티티에 대한 index를 진행할 때, OAuth2 로그인와 같은 특수한 상황이 아닌경우 멤버 id를 통해서만 멤버를 search하기 위함

To-Do

  • entitiy 에서 email 제거
  • Reslover 수정
  • Controller에서 이메일을 받는 것을 memberId로 수정

ETC

굳이 잘 돌아가는 코드를 손 대는거 아닌가?
그리고 지금 FCM 브랜치 풀리퀘 못한 상태라 리팩토링은 좀 그렇다.
일단 테스트 코드나 좀 더 짜봐야겠다

@JunRain2 JunRain2 added the backend backend label Apr 1, 2024
@JunRain2 JunRain2 closed this as completed May 7, 2024
@JunRain2 JunRain2 self-assigned this Jun 8, 2024
@JunRain2 JunRain2 reopened this Jun 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend backend
Projects
None yet
Development

No branches or pull requests

1 participant