novuhq / novu

Open-Source Notification Platform. Embeddable Notification Center, E-mail, Push and Slack Integrations.
https://novu.co
Other
35.44k stars 3.92k forks source link

🐛 Bug Report: Incompatibility between nest-swagger-ui and @novu/framework #6994

Open paulwer opened 1 week ago

paulwer commented 1 week ago

📜 Description

When the swagger explorer tries to check the controller of the nest integration and their metadata, it failes with the following error: image

👟 Reproduction steps

NestJs Repository with additional dependency: @nestjs/swagger & swagger-ui-express

I guess the origin of this error is maybe, that decorator data is not exported at build process. (but I am not sure to be honest)

👍 Expected behavior

Should be able to fetch metadata from the classes and reuse them.

👎 Actual Behavior with Screenshots

image

Novu version

@novu/framework@2.5.0

npm version

7

node version

18

📃 Provide any additional context for the Bug.

Potential fix: https://stackoverflow.com/questions/58596139/nestjs-swagger-issue

Use emitDecoratorMetadata in tsconfig.json

👀 Have you spent some time to check if this bug has been raised before?

🏢 Have you read the Contributing Guidelines?

Are you willing to submit PR?

None

linear[bot] commented 1 week ago

NV-4753 🐛 Bug Report: Incompatibility between nest-swagger-ui and @novu/framework

paulwer commented 1 week ago

image