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

Fastify install is required despite Express as the selected platform #3

Open
KerryRitter opened this issue Oct 30, 2020 · 4 comments
Open

Comments

@KerryRitter
Copy link

I installed and setup as specified, but got the following error on start:

node_modules/@nestifyjs/cookie/lib/interceptors/fastify-cookie.interceptor.d.ts:4:26 - error TS2307: Cannot find module 'fastify' or its corresponding type declarations.

4 import * as fastify from 'fastify';
                           ~~~~~~~~~
node_modules/@nestifyjs/cookie/lib/interceptors/fastify-cookie.interceptor.d.ts:5:46 - error TS2307: Cannot find module 'fastify' or its corresponding type declarations.

5 import { FastifyReply, FastifyRequest } from 'fastify';
                                               ~~~~~~~~~

I installed fastify as a dev dependency to get the compilation to work, but was splattered with warnings (due to a version mismatch I assume).

@kayun
Copy link
Contributor

kayun commented Oct 30, 2020

Hello! Thanks for sharing the problem!

What version of nestjs did you use? I just checked on version 6 and everything works fine there. Express platform works correctly, without the need to install a Fastify. There are also no warnings on the Fastify platform.

I will be glad to solve the problem, Artem.

@KerryRitter
Copy link
Author

I am on a fresh install of Nest 7, latest CLI and framework.

@kayun
Copy link
Contributor

kayun commented Oct 30, 2020

It seems that the problems are really in the new version of nestjs. I will try to deal with this problem at the weekend.

@KerryRitter
Copy link
Author

Awesome! Thank you for your efforts :) I'm using cookie-parser for now, but I'll circle back whenever you get time to push a fix. Much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants