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

Update example with-sentry #24819

Merged
merged 7 commits into from
May 7, 2021
Merged

Update example with-sentry #24819

merged 7 commits into from
May 7, 2021

Conversation

iker-barriocanal
Copy link
Contributor

The official Sentry Next.js SDK is now the recommended choice to use with Next.js, instead of the previous workarounds. This PR updates the example, which now uses the SDK.

@ijjk ijjk added the examples Issue/PR related to examples label May 5, 2021
@vitalybaev
Copy link
Contributor

I believe the following notice should be provided

Early Adopter

The SDK is in the early stages of being released. This means changes are possible and may prevent the SDK from working as expected. We recognize the irony.

@leerob
Copy link
Member

leerob commented May 5, 2021

Please see #20759.

@leerob leerob closed this May 5, 2021
@iker-barriocanal
Copy link
Contributor Author

Hi @leerob, the SDK is not using the plugin system anymore, see getsentry/sentry-javascript#3462.

@leerob leerob reopened this May 6, 2021
@kodiakhq kodiakhq bot merged commit 4bc87e4 into vercel:canary May 7, 2021
@iker-barriocanal iker-barriocanal deleted the update-example-withsentry branch May 7, 2021 13:06
flybayer pushed a commit to blitz-js/next.js that referenced this pull request Jun 1, 2021
The [official Sentry Next.js SDK](https://docs.sentry.io/platforms/javascript/guides/nextjs/) is now the recommended choice to use with Next.js, instead of the previous workarounds. This PR updates the example, which now uses the SDK.
@vercel vercel locked as resolved and limited conversation to collaborators Jan 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue/PR related to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants