Open API doc not generated for PartialType DTO

[kalaivanan-muthusamy]

Bug Report

When using the PartialType for DTO, it is not reflected in the OpenAPI documentation.
https://docs.nestjs.com/openapi/mapped-types

Input Code

export class CreateUserDTO {
  @IsNotEmpty()
  @IsString()
  name: string;
}

export class UpdateUserDTO extends PartialType(CreateUserDTO) {
}

Current behavior

UpdateUserDTO schema in the OpenAPI documentation is missing the name property

Expected behavior

OpenAPI document to have name: string in UpdateUserDTO Schema. But missing the name property in the generated schema.

Environment

Nest version: 7.0.0

 
For Tooling issues:
- Node version: 12.18.1
- Platform: Windows

Others:

[jmcdo29]

Please provide a minimum reproduction repository.

[kalaivanan-muthusamy]

Sample Repo: https://github.com/kalaivanan-muthusamy/nestjs-swagger-issue-1074

I am expecting the properties in CreateUserDto should also exist in UpdateUserDto as optional. But the properties are not present in the schema

[kamilmysliwiec]

Import PartialType from @nestjs/swagger

[kalaivanan-muthusamy]

I missed it completely. It is clearly mentioned in the document. Thanks for the wonderful document.

This should be the issue with cli then. As CLI is importing the PartialType from @nestjs/mapped-types instead @nestjs/swagger.

Opened the PR for the expected change
nestjs/schematics#545

[ZeeshanAhmadKhalil]

But it's not Making the properties in StarterDto required.

StarterDto

export class StarterDto {

    @ApiProperty({
        required: true,
    })
    @IsNotEmpty()
    recipientName: string

    @ApiProperty({
        required: false
    })
    recipientPronounce?: string

    @ApiProperty({
        type: 'string',
        format: 'binary',
        required: false,
    })
    recipientNameFile?: any

    @ApiProperty({
        required: true,
    })
    @IsNotEmpty()
    recipientRelationship: string

    @ApiProperty({
        required: true,
    })
    @IsNotEmpty()
    mentionRecipientInSong: boolean

    @ApiProperty({
        required: true,
    })
    @IsNotEmpty()
    occasion: string

    @ApiProperty({
        required: true,
    })
    @IsNotEmpty()
    mentionOccasionInSong: boolean
}

UpdateStarterDto

import { PartialType } from '@nestjs/swagger';

export class UpdateStarterDto extends PartialType(StarterDto) {

    @ApiProperty()
    @IsNotEmpty()
    orderId: string
}