nestjs / swagger

OpenAPI (Swagger) module for Nest framework (node.js) :earth_americas:
https://nestjs.com
MIT License
1.69k stars 474 forks source link

Expand extension support to Document and Schema objects #2683

Closed drewish closed 1 year ago

drewish commented 1 year ago

Is there an existing issue that is already proposing this?

Is your feature request related to a problem? Please describe it

In https://github.com/nestjs/swagger/issues/195 partial support for extensions was added, but it didn't apply to some other places you'd commonly want to use it.

Describe the solution you'd like

Extend https://github.com/nestjs/swagger/issues/195 to apply to the DocumentBuilder and Schema objects, rather than just Operations.

Teachability, documentation, adoption, migration strategy

No response

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

Trying to use Redocly's extensions on the top level Document (for x-tag-groups) and Schema (for x-tags) objects.

kamilmysliwiec commented 1 year ago

Let's track this here https://github.com/nestjs/swagger/pull/2684

drewish commented 1 year ago

Yeah turned out to be easier than I thought to get the PR working.