fix: set default value of createNotice dto

gsainfoteam · Nov 22, 2023 · 499d4f3 · 499d4f3
1 parent 1beacc6
commit 499d4f3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/notice/dto/createNotice.dto.ts b/src/notice/dto/createNotice.dto.ts
@@ -1,10 +1,10 @@
 import { Type } from 'class-transformer';
 import {
-  IsString,
-  IsOptional,
-  IsNumber,
-  IsNotEmpty,
   IsDate,
+  IsNotEmpty,
+  IsNumber,
+  IsOptional,
+  IsString,
   MaxLength,
 } from 'class-validator';
 
@@ -25,9 +25,9 @@ export class CreateNoticeDto {
 
   @IsNumber({}, { each: true })
   @IsOptional()
-  tags?: number[];
+  tags: number[] = [];
 
   @IsString({ each: true })
   @IsOptional()
-  images?: string[];
+  images: string[] = [];
 }
diff --git a/src/notice/notice.service.ts b/src/notice/notice.service.ts
@@ -65,7 +65,7 @@ export class NoticeService {
     { title, body, deadline, tags, images }: CreateNoticeDto,
     userUuid: string,
   ) {
-    if (images) {
+    if (images.length) {
       await this.imageService.validateImages(images);
     }