nestjs-sentry icon indicating copy to clipboard operation
nestjs-sentry copied to clipboard

NestJs 9 Support

Open badsgahhl opened this issue 2 years ago • 7 comments

Currently the package is requiring NestJs 8 as peer dependency. You can only install the extension via a force install. The package should therefor be updated to work with the current Nest Version

badsgahhl avatar Jul 08 '22 18:07 badsgahhl

I've added NestJS 9 & sentry 7.7.0 in this PR https://github.com/ntegral/nestjs-sentry/pull/93

I've also created a fork-package in the mean time. https://www.npmjs.com/package/@xiifain/nestjs-sentry

xiifain avatar Jul 21 '22 17:07 xiifain

Thanks @xiifain .

@Team any plans to merge this to the main branch any sooner?

kannas14 avatar Aug 08 '22 06:08 kannas14

People, please make this happen. It's a blocker for our update to Nest 9.

martinsotirov avatar Aug 08 '22 08:08 martinsotirov

There is some update? It would be nice to have this merged, do you know if this library is still maintained?

AlbertHernandez avatar Aug 17 '22 13:08 AlbertHernandez

Last commit was back in January - @ntegral are you around or should we migrate to a fork?

zackdotcomputer avatar Sep 01 '22 09:09 zackdotcomputer

In the meantime, I've created a fork as well - https://github.com/travelerdev/nestjs-sentry

It upgrades to NestJS 9.x support, as well as Sentry 7.x and separates out dependencies on @nestjs/graphql into their own package so that you can add Sentry to Nest without necessarily adding all the graphql-related dependencies (since optionalDependencies doesn't actually work the way this repository uses it - see #71 )

zackdotcomputer avatar Sep 01 '22 12:09 zackdotcomputer

Any progress on this?

Edit: I see the PR is merged, thanks

slaveofcode avatar Sep 14 '22 17:09 slaveofcode