MappedType PartialType throws error when using option decorator factory input about graphql HOT 7 CLOSED

Got the same issue, had to downgrade from 12.0.10 to 12.0.9. I can test if any beta versions become available

from graphql.

Please provide a minimum reproduction repository (Git repository/StackBlitz/CodeSandbox project).

from graphql.

After poking around, it looks like this recent change broke the validation. The following will help, utilizing the new PartialTypeOptions interface, but I need to dive into this more to understand why that change caused the previously working validation to fail.

Before:

@InputType()
export class UpdateDeviceSettingsInput extends PartialType(DeviceSettings, InputType) {}

After:

@InputType()
export class UpdateDeviceSettingsInput extends PartialType(DeviceSettings, { decorator: InputType, omitDefaultValues: true }) {}

from graphql.

@kamilmysliwiec @cskiwi tagging you all just for reference.

from graphql.

@jkossis would you like to create a PR to fix this issue? My hunch is that the current behavior of isPartialTypeOptions might be broken but I'd have to double-check to confirm

from graphql.

Thank you @jkossis. Was traveling abroad. Sorry for the late response. So a minimum reporduction is not necessary anymore?

from graphql.

Version 12.0.11 resolves this issue for me. thanks for the fix!

from graphql.