Skip to content
This repository has been archived by the owner on Jul 8, 2021. It is now read-only.

Issues with Nest 6.0 #1

Closed
kjetilhp opened this issue Mar 29, 2019 · 2 comments · Fixed by #2
Closed

Issues with Nest 6.0 #1

kjetilhp opened this issue Mar 29, 2019 · 2 comments · Fixed by #2
Assignees
@mentos1386
Labels
bug Something isn't working

Comments

@kjetilhp
Copy link

Seems to be an issues after upgrading to nest 6

TS2416: Property 'intercept' in type 'AbstractMorganInterceptor' is not assignable to the same property in base type 'NestInterceptor<any, any>'.
  Type '(context: ExecutionContext, call$: Observable<any>) => Observable<any>' is not assignable to type '(context: ExecutionContext, next: CallHandler<any>) => Observable<any> | Promise<Observable<any>>'.
    Types of parameters 'call$' and 'next' are incompatible.
      Type 'CallHandler<any>' is missing the following properties from type 'Observable<any>': _isScalar, source, operator, lift, and 6 more.
@mentos1386 mentos1386 added the bug Something isn't working label Mar 29, 2019
@mentos1386 mentos1386 self-assigned this Mar 29, 2019
@mentos1386
Copy link
Owner

@kjetilhp Thanks for reporting this! I will update it this afternoon and release a new version.

mentos1386 added a commit that referenced this issue Apr 5, 2019
@mentos1386
fix: upgraded to nestjs v6
d627af0 
BREAKING CHANGE: not compatible with nestjs v5 and bellow

fix #1
@mentos1386 mentos1386 mentioned this issue Apr 5, 2019
Merged
mentos1386 added a commit that referenced this issue Apr 5, 2019
@mentos1386
fix: upgraded to nestjs v6
ceba8f5 
BREAKING CHANGE: not compatible with nestjs v5 and bellow

fix #1
@mentos1386
Copy link
Owner

mentos1386 commented Apr 5, 2019 •
edited
Loading

@kjetilhp Sorry it took me so long to finish this. Released in version 1.0.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mentos1386 mentos1386

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: upgraded to nestjs v6 mentos1386/nest-morgan
2 participants
@kjetilhp @mentos1386