Jackie McDoniel

icon indicating a website link https://jaymcdoniel.dev icon indicating an email [email protected]

icon location Concrete, WA icon location I'm a full-stack developer with a passion for @nestjs and @nrwl. I love working with TypeScript and learning new ways to tackle common problems.

Results 231 comments of Jackie McDoniel

chore(deps): bump fastify from 4.7.0 to 4.8.1

@kamilmysliwiec as there's a security vulnerability to this, do you think we can get it merged in soon?

Dynamic modules with forwardRef in imports doesn't allow app to start

Okay, so this works without the `forwardRef`, and your reproduction shows that using a `forwardRef` in the dynamic module, does indeed, cause this error. Can you explain your use case...

Ability to define the max body size using property in NestApplicationOptions

I like the idea for it! I'd say go ahead and make a PR, the API looks clean enough. Make sure it works for fastify and express please

Nest 9.1.3 version does not work with webpack config from the serverless page

Ah, yeah, this would be due to the new `require` I added in. @kamilmysliwiec does your `loadRequire()` bypass webpack's replacement, or is this something we'll need to update in the...

feat(core): free port listening

By the way, using `0` as the port means that the OS will find a free port for you. I do this in just about every e2e test that I...

feat(core): free port listening

Also, what would be the reason for keeping a port within a certain range, like under 4000? So long as it's above port 1023 is should _generally_ be okay as...

NestJS 9 with Fastify ignores @Res() in controller

This actually might be related to something I'm looking into with `@fastify/view` integration and not being able to `res.view()` from an exception filter. I think there's somewhere that an `await`...

NestJS 9 with Fastify ignores @Res() in controller

> Did you have a chance to find the root cause @jmcdo29? Let me know and I can take over if you're busy raised_hands @kamilmysliwiec that might be best. What...

After upgrading form v.0.27.2 to v.1.1.2 Axios not working in Nestjs

I just want to point out that this is **not** a problem with Axios's new version **nor** is it a problem with how Nest uses Axios. Nest has `@nestjs/axios` for...

After upgrading form v.0.27.2 to v.1.1.2 Axios not working in Nestjs

@jasonsaayman that may be, and thank you for working on fixing that up. This is just the second time I've seen someone go to a _library_ that recently went through...