[BE] 유저 닉네임 변경 API 추가 #223
Conversation
답답해서 한 건 아니잖아요 ㅋㅋㅋㅋ 어제 저희끼리 모여있을 때 제가 할게요~ 했던 거잖아요ㅋㅋㅋㅋ!!! 아니 왜 그러세요!! |
앗 ㅋㅋㅋ 이거 말고 그 default 닉네임이요 ㅎㅎㅎㅎㅎㅎㅎㅎㅎ 수정 사항이 같이 적용되서 보이네요 |
BE/src/auth/user.controller.ts
Outdated
| @@ -27,4 +29,20 @@ export class UserController { | |||
| getProfile(@Req() request: Request) { | |||
| return this.userService.getProfile(parseInt(request.user.userId, 10)); | |||
| } | |||
|
|
|||
| @Post('/rename') | |||
There was a problem hiding this comment.
🟢 닉네임 수정은 patch 메소드가 더 낫지 않을까용...? 어떻게 생각하세요?
There was a problem hiding this comment.
오 완전 좋아요 patch!!
맞아요.. 당연한건데.. 전체 수정이 아니라 부분 수정이니까!!
당장 반영하겠습니다.
BE/src/auth/user.service.ts
Outdated
| const isDuplicated = await this.userRepository.findBy({ | ||
| nickname: newName, | ||
| }); | ||
| if (isDuplicated.length > 0) { | ||
| throw new BadRequestException('이미 존재하는 닉네임입니다.'); | ||
| } |
There was a problem hiding this comment.
🟢 existsBy 이런거로 boolean 값 받아서 할 수도 있을 것 같아요!
There was a problem hiding this comment.
오우..! 저도 저런 식으로만 했어서 생각도 못했네요 그런 메소드가 있었군요..! 감사합니다!
There was a problem hiding this comment.
안그래도 boolean 으로 하고 싶었는데 저렇게 찾으니까 없는 경우에도 빈 배열이어서.. 뭐 어떤 상황에서도 다 true로 가지고 오더라구요..ㅋㅋㅋ
감사합니다!!!
✅ 주요 작업
💭 고민과 해결과정