Merge pull request #30 from oodd-team/OD-33

유저 정보 수정 기능 수정

Author: happbob

src/user/dto/patch-user.request.ts

@@ -1,16 +1,57 @@
-import { IsOptional, IsString, IsUrl, MaxLength } from 'class-validator';
+import {
+  IsEmail,
+  IsNotEmpty,
+  IsOptional,
+  IsString,
+  IsUrl,
+  MaxLength,
+} from 'class-validator';
 import { ApiProperty } from '@nestjs/swagger';
 
 export class PatchUserRequest {
+  @IsOptional()
+  @IsString()
+  @MaxLength(100, { message: '이름은 최대 100자까지 입력할 수 있습니다.' })
+  @IsNotEmpty({ message: '이름을 비울 수 없습니다.' })
+  @ApiProperty({
+    description: '수정할 이름',
+    example: '새 이름',
+    required: false,
+  })
+  name?: string;
+
+  @IsOptional()
+  @IsString()
+  @MaxLength(15, { message: '전화번호는 최대 15자까지 입력할 수 있습니다.' })
+  @IsNotEmpty({ message: '전화번호를 비울 수 없습니다.' })
+  @ApiProperty({
+    description: '수정할 전화번호',
+    example: '010-9876-5432',
+    required: false,
+  })
+  phoneNumber?: string;
+
+  @IsOptional()
+  @IsEmail()
+  @MaxLength(100, { message: '이메일은 최대 100자까지 입력할 수 있습니다.' })
+  @IsNotEmpty({ message: '이메일을 비울 수 없습니다.' })
+  @ApiProperty({
+    description: '수정할 이메일',
+    example: '[email protected]',
+    required: false,
+  })
+  email?: string;
+
   @IsOptional()
   @IsString()
   @MaxLength(10, { message: '닉네임은 최대 10자까지 입력할 수 있습니다.' })
+  @IsNotEmpty({ message: '닉네임을 비울 수 없습니다.' })
   @ApiProperty({
     description: '수정할 닉네임',
-    example: '새닉네임',
+    example: '새 닉네임',
     required: false,
   })
-  nickname?: string | null;
+  nickname?: string;
 
   @IsOptional()
   @IsUrl({}, { message: '유효한 URL 형식이어야 합니다.' })
@@ -19,7 +60,7 @@ export class PatchUserRequest {
     example: 'https://example.com/profile.jpg',
     required: false,
   })
-  profilePictureUrl?: string | null;
+  profilePictureUrl?: string;
 
   @IsOptional()
   @IsString()
@@ -29,5 +70,5 @@ export class PatchUserRequest {
     example: '소개글~~~^^',
     required: false,
   })
-  bio?: string | null;
+  bio?: string;
 }

src/user/dto/patch-user.response.ts

@@ -8,19 +8,37 @@ export class PatchUserResponse {
   userId: number;
 
   @ApiProperty({
-    description: '수정된 닉네임',
+    description: '수정 후 이름',
+    example: '새 이름',
+  })
+  name: string;
+
+  @ApiProperty({
+    description: '수정 후 전화번호',
+    example: '010-9876-5432',
+  })
+  phoneNumber: string;
+
+  @ApiProperty({
+    description: '수정 후 이메일',
+    example: '[email protected]',
+  })
+  email: string;
+
+  @ApiProperty({
+    description: '수정 후 닉네임',
     example: '새 닉네임',
   })
   nickname: string;
 
   @ApiProperty({
-    description: '수정된 프로필 사진 URL',
+    description: '수정 후 프로필 사진 URL',
     example: 'https://example.com/profile.jpg',
   })
   profilePictureUrl: string;
 
   @ApiProperty({
-    description: '수정된 자기소개',
+    description: '수정 후 자기소개',
     example: '소개글~~~^^',
   })
   bio: string;

src/user/user.controller.ts

@@ -62,6 +62,9 @@ export class UserController {
 
     return new BaseResponse<PatchUserResponse>(true, '유저 정보 수정 성공', {
       userId: updatedUser.id,
+      name: updatedUser.name,
+      phoneNumber: updatedUser.phoneNumber,
+      email: updatedUser.email,
       nickname: updatedUser.nickname,
       profilePictureUrl: updatedUser.profilePictureUrl,
       bio: updatedUser.bio,

src/user/user.service.ts

@@ -64,15 +64,11 @@ export class UserService {
     });
 
     try {
-      if (patchUserRequest.nickname !== undefined) {
-        user.nickname = patchUserRequest.nickname;
-      }
-      if (patchUserRequest.profilePictureUrl !== undefined) {
-        user.profilePictureUrl = patchUserRequest.profilePictureUrl;
-      }
-      if (patchUserRequest.bio !== undefined) {
-        user.bio = patchUserRequest.bio;
-      }
+      Object.entries(patchUserRequest).forEach(([key, value]) => {
+        if (value !== undefined) {
+          user[key] = value;
+        }
+      });
 
       return await this.userRepository.save(user);
     } catch (error) {