elysiajs / elysia-swagger

A plugin for Elysia to auto-generate Swagger page

MIT License

90 stars 46 forks source link

fix: swagger date time #148

Closed Abraxas-365 closed 1 month ago

Abraxas-365 commented 2 months ago

124 Fix date time type issue

sergaks commented 2 months ago

please approve this )

dodas commented 1 week ago

Does this actually do anything? It seems that here stringifiedSwaggerOptions is being JSON.parsed and from there on being treated as OpenAPIV3.Document, even though it's likely a subset of SwaggerUIOptions. These two interfaces have no properties in common. So the condition swaggerOptions.components && swaggerOptions.components.schemas likely never evaluates to true.

dodas commented 4 days ago

cc @marclave @Abraxas-365