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(*): Member의 id를 Security에서 가져오도록 변경 #32

Merged
merged 4 commits into from
Jul 26, 2024

Conversation

morenow98
Copy link
Member

PR 변경된 내용

  • Contoller에서 memberId = "1" 과 같은 식으로 하드코딩되어 있는 것을 SecurityContext에서 가져오도록 변경

추가 내용

  • Member의 validate 테스트 코드 구현
  • 에러 코드가 잘못 올라가 있었던 오류 수정

참조

Closes #30

@morenow98 morenow98 requested a review from InHyeok-J July 26, 2024 06:13
@morenow98 morenow98 self-assigned this Jul 26, 2024
@morenow98 morenow98 merged commit 44e4ea1 into develop Jul 26, 2024
1 check passed
@morenow98 morenow98 deleted the feature/30-get-member-from-security branch July 26, 2024 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Member의 id를 Security에서 가져오도록 변경
2 participants