Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to NestJS 8 #40

Open
talski opened this issue Jul 21, 2021 · 5 comments
Open

Update to NestJS 8 #40

talski opened this issue Jul 21, 2021 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@talski
Copy link

talski commented Jul 21, 2021

No description provided.

@sinnrrr
Copy link

sinnrrr commented Jul 24, 2021

+1, have the following error:

Argument of type 'import("C:/Users/dimas/Work/new/evoraid/node_modules/@nestjs/common/interfaces/nest-application.interface").INestApplication' is not assignable to parameter of type 'import("C:/Users/dimas/Work/new/evoraid/node_modules/nestjs-redoc/node_modules/@nestjs/common/interfaces/nest-application.interface").INestApplication'.   The types returned by 'startAllMicroservices(...)' are incompatible between these types.     Type 'Promise<INestApplication>' is missing the following properties from type 'INestApplication': use, enableCors, listen, getUrl, and 21 more.

@john526
Copy link

john526 commented Aug 2, 2021

Hello @sinnrrr, uninstall nestjs-redoc and install npm i @ulrichf/nestjs-redoc it work for me

@ojoanalogo ojoanalogo added the bug Something isn't working label Aug 2, 2021
@ojoanalogo ojoanalogo self-assigned this Aug 2, 2021
@UlrichFrancomme
Copy link
Contributor

Hello @mxarc, I made a PR (#42 ) for this issue. Could you take a look please?

@john526 Thank you for using my package! 😄 I do no intend to keep it updated, it is only to help until this issue is solved. So I suggest to use mxarc's package again as soon as the PR will be merged.

@sinnrrr
Copy link

sinnrrr commented Aug 3, 2021

Hello @sinnrrr, uninstall nestjs-redoc and install npm i @ulrichf/nestjs-redoc it work for me

hey, thanks for suggestion
apparently, @UlrichFrancomme have made a PR, solving this issue
waiting for @mxarc to approve it

@ojoanalogo
Copy link
Owner

Package updated on npm registry, thanks @UlrichFrancomme !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants