implement update user

Author: kentloog

src/users/dto/update-user.dto.ts

@@ -0,0 +1,25 @@
+import { Gender } from './../../shared/enum/gender';
+import { ApiModelProperty } from '@nestjs/swagger';
+import { IsOptional, IsString, IsEnum, IsISO8601 } from 'class-validator';
+
+export class UpdateUserDto {
+    @ApiModelProperty()
+    @IsOptional()
+    @IsString()
+    readonly firstName: string;
+
+    @ApiModelProperty()
+    @IsOptional()
+    @IsString()
+    readonly lastName: string;
+
+    @ApiModelProperty()
+    @IsOptional()
+    @IsEnum(Gender)
+    readonly gender: Gender;
+
+    @ApiModelProperty()
+    @IsOptional()
+    @IsISO8601()
+    readonly birthday: string;
+}

src/users/users.controller.ts

@@ -8,13 +8,15 @@ import {
     Delete,
     Req,
     UseGuards,
+    Put,
 } from '@nestjs/common';
 import { CreateUserDto } from './dto/create-user.dto';
 import { UsersService } from './users.service';
 import { UserDto } from './dto/user.dto';
 import { ApiUseTags, ApiOkResponse, ApiBearerAuth } from '@nestjs/swagger';
 import { UserLoginResponseDto } from './dto/user-login-response.dto';
 import { AuthGuard } from '@nestjs/passport';
+import { UpdateUserDto } from './dto/update-user.dto';
 
 @Controller('users')
 @ApiUseTags('users')
@@ -32,7 +34,7 @@ export class UsersController {
     @Post('login')
     @HttpCode(200)
     @ApiOkResponse({ type: UserLoginResponseDto })
-    async login(
+    login(
         @Body() userLoginRequestDto: UserLoginRequestDto,
     ): Promise<UserLoginResponseDto> {
         return this.usersService.login(userLoginRequestDto);
@@ -46,11 +48,22 @@ export class UsersController {
         return this.usersService.findAll();
     }
 
+    @Put('me')
+    @ApiBearerAuth()
+    @UseGuards(AuthGuard('jwt'))
+    @ApiOkResponse({ type: UserDto })
+    update(
+        @Body() updateUserDto: UpdateUserDto,
+        @Req() request,
+    ): Promise<UserDto> {
+        return this.usersService.update(request.user.id, updateUserDto);
+    }
+
     @Delete()
     @ApiBearerAuth()
     @UseGuards(AuthGuard('jwt'))
     @ApiOkResponse({ type: UserDto })
-    async delete(@Req() request): Promise<UserDto> {
-        return await this.usersService.delete(request.user.id);
+    delete(@Req() request): Promise<UserDto> {
+        return this.usersService.delete(request.user.id);
     }
 }

src/users/users.service.ts

@@ -7,6 +7,7 @@ import { CreateUserDto } from './dto/create-user.dto';
 import { UserLoginResponseDto } from './dto/user-login-response.dto';
 import { JwtPayload } from './auth/jwt-payload.model';
 import { sign } from 'jsonwebtoken';
+import { UpdateUserDto } from './dto/update-user.dto';
 
 @Injectable()
 export class UsersService {
@@ -87,6 +88,25 @@ export class UsersService {
         return new UserLoginResponseDto(user, token);
     }
 
+    async update(id: string, updateUserDto: UpdateUserDto): Promise<UserDto> {
+        const user = await this.usersRepository.findByPk<User>(id);
+        if (!user) {
+            throw new HttpException('User not found.', HttpStatus.NOT_FOUND);
+        }
+
+        user.firstName = updateUserDto.firstName || user.firstName;
+        user.lastName = updateUserDto.lastName || user.lastName;
+        user.gender = updateUserDto.gender || user.gender;
+        user.birthday = updateUserDto.birthday || user.birthday;
+
+        try {
+            const data = await user.save();
+            return new UserDto(data);
+        } catch (err) {
+            throw new HttpException(err, HttpStatus.INTERNAL_SERVER_ERROR);
+        }
+    }
+
     async delete(id: string): Promise<UserDto> {
         const user = await this.usersRepository.findByPk<User>(id);
         await user.destroy();