Merged PR 661: APIバリデータの漏れを修正

## 概要
[Task3402: APIバリデータの漏れを修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3402)

- Optinal指定が必要なパラメータにOptionalが抜けていた箇所を修正
  - 以前はバリデータ自体が無かったので、IsOptionalが抜けていても偶然動いていた
- 他requestにも同様の漏れがないか `?:` で検索したが、この1件のみだと思われる

## レビューポイント
- 他に同様の記載はなさそうか
- `?:` による省略可能表記以外の記法で省略パラメータは定義されていない認識だが、合っているか

## 動作確認状況
- npm run testは通過
This commit is contained in:
湯本 開 2023-12-25 07:34:49 +00:00
parent f6c3f69801
commit f2ef2228c3

View File

@ -1,5 +1,5 @@
import { ApiProperty } from '@nestjs/swagger'; import { ApiProperty } from '@nestjs/swagger';
import { IsBoolean, IsEmail, IsIn, IsInt, MaxLength } from 'class-validator'; import { IsBoolean, IsEmail, IsIn, IsInt, IsOptional, MaxLength } from 'class-validator';
import { import {
TASK_LIST_SORTABLE_ATTRIBUTES, TASK_LIST_SORTABLE_ATTRIBUTES,
USER_LICENSE_STATUS, USER_LICENSE_STATUS,
@ -291,6 +291,7 @@ export class UpdateAcceptedVersionRequest {
acceptedPrivacyNoticeVersion: string; acceptedPrivacyNoticeVersion: string;
@ApiProperty({ description: '更新バージョンDPA', required: false }) @ApiProperty({ description: '更新バージョンDPA', required: false })
@MaxLength(255) @MaxLength(255)
@IsOptional()
acceptedDPAVersion?: string; acceptedDPAVersion?: string;
} }