Please Add `optional: boolean` to @Query route param decorator

I’m submitting a…

[x] Feature request

Current behavior

Using @Query('foo') will always make a query parameter required.

Expected behavior

The Query decorator should have an optional: boolean property available, similar to how the Args decorator for gql resolvers has nullable and other options.

e.g. @Query({ name: 'foo', optional: true })

I’m ok with it always being a string and then using pipes to validate / transform the value downstream but at least nullable or optional, whatever, would be nice.

Minimal reproduction of the problem with instructions

NA

What is the motivation / use case for changing the behavior?

I’m creating a controller and would like to have optional parameters with sensible defaults.

Environment

NA

Extra

I believe this is the correct location in code: https://github.com/nestjs/nest/blob/master/packages/common/decorators/http/route-params.decorator.ts#L95

[hwalyonglance]

@justinmchase

@ApiImplicitQuery({
	name: 'foo',
	required: false,
	type: String
})
myMethod(@Query('foo') foo: string){

}

[kamilmysliwiec]

Both solutions are fine 😃