fix required field

78bd3a51 · nanahira · 47da8a85 · 78bd3a51
Commit 78bd3a51 authored Nov 10, 2023 by nanahira
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

src/decorators/property.ts src/decorators/property.ts +5 -2

No files found.
--- a/src/decorators/property.ts
+++ b/src/decorators/property.ts
@@ -36,7 +36,7 @@ function swaggerDecorator(
 ) {
  return ApiProperty({
    default: options.default,
-    required: options.required && options.default == null,
+    required: !!(options.required && options.default == null),
    example: options.default,
    description: options.description,
    ...injected,
@@ -192,6 +192,9 @@ export const NotColumn = (
 ): PropertyDecorator =>
  MergePropertyDecorators([
    Exclude(),
-    swaggerDecorator(options),
+    swaggerDecorator({
+      required: false,
+      ...options,
+    }),
    Metadata.set('notColumn', true, 'notColumnFields'),
  ]);