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

NestJs 9 Support #91

Open
badsgahhl opened this issue Jul 8, 2022 · 7 comments
Open

NestJs 9 Support #91

badsgahhl opened this issue Jul 8, 2022 · 7 comments

Comments

@badsgahhl
Copy link

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

@xiifain
Copy link
Contributor

xiifain commented Jul 21, 2022

I've added NestJS 9 & sentry 7.7.0 in this PR #93

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

@kannas14
Copy link

kannas14 commented Aug 8, 2022

Thanks @xiifain .

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

@martinsotirov
Copy link

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

@AlbertHernandez
Copy link

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

@zackdotcomputer
Copy link

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

@zackdotcomputer
Copy link

zackdotcomputer commented Sep 1, 2022

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 )

@slaveofcode
Copy link

slaveofcode commented Sep 14, 2022

Any progress on this?

Edit: I see the PR is merged, thanks

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

7 participants