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

[NB-187] api에서 유저 식별자 변경 #36

Merged
merged 7 commits into from
Aug 5, 2024

Conversation

liljoon
Copy link
Contributor

@liljoon liljoon commented Aug 2, 2024

개요

NB-187:

  • 유저 식별자 userId -> nickname 으로 변경
  • 이에 따른 테스트 코드 수정
  • validation을 getUserByNickname을 통해 자동으로 처리

PR 유형

어떤 변경 사항이 있나요?

  • 버그 수정
  • 코드 리팩토링
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고 (Ctrl + 클릭하세요.)
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

liljoon added 6 commits August 2, 2024 17:31
- userId -> nickname으로 식별자 변경하여 구현
- userId -> nickname으로 식별자 변경하여 구현
- userId -> nickname으로 수정하여 호출
- 테스트 메소드명 변경
- userId -> nickname 으로 표기
@liljoon liljoon requested a review from Aram-su August 2, 2024 09:12
Copy link
Contributor

@Aram-su Aram-su left a comment

Choose a reason for hiding this comment

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

User 조회 Id 에서 Nickname으로 수정 된 것 확인했습니다

- 체이닝을 이용하여 코드 간소화
@liljoon liljoon merged commit 4ecbcc6 into dev Aug 5, 2024
1 check passed
@liljoon liljoon deleted the fix/replace-userid-with-nickname branch August 5, 2024 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants