-
Notifications
You must be signed in to change notification settings - Fork 1
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
PATCH api/users 기능 추가 #8
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Query Params 건 때문에 Request changes 로 남깁니다 빠샤 💪
src/api/users/users.service.ts
Outdated
return this.prismaService.user.findUnique({ | ||
where: { username }, | ||
}); | ||
} | ||
|
||
update(id: string, updateUserDto: UpdateUserDto) { | ||
return `This action updates a #${id} user`; | ||
async isDateOnlyFormatValid(dateOnly: boolean, dates: string[]) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 얘는 private method 로 만들면 좋을 것 같고,
- private method 는 맨 위든 맨 밑이든 한 쪽으로 몰면 좋을 것 같아요!
코드 리뷰 기반하여 수정 완료했습니다.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다 👏
@@ -56,7 +70,7 @@ export class UsersService { | |||
if (badRequestErros.length > 0) { | |||
throw new BadRequestException(badRequestErros); | |||
} | |||
|
|||
console.log(roomCode); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
나중에 시간 되면 이런 테스트 코드는 삭제시켜주시면 좋을듯!
PATCH api/users 기능 추가하였습니다.
요청 형식은
query params
roomCode
Body
username
dateOnly
dates
입니다.