-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Next.js mjs setup issue #662
Comments
Assigning to @getsentry/support for routing ⏲️ |
@TheCukitoDev are you referring to the cjs tab being selected by default in the docs? |
also but the main issue is while executing the wizzard doesen't cares about the project js specification. It always install the CJS version instead of the ES6 that nextjs uses. |
@TheCukitoDev you are right, if no previous config was found we create a js file by default as this works with the majority of Next.js versions. We have this on our backlog 👍 |
Core or SDK?
Platform/SDK
Which part? Which one?
Next.js Docs (https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/)
Description
Next.js comes by default with support only for ES6 so the mjs should be the default. Also I think while installing sentry with the wizard should install the ES6 version instead of the cjs version
Suggested Solution
Install the ES6 version instead of the cjs version or add a function that detects what version uses only reading package.json select what version install. Like if it says type: module install the es6 version or in the CLI ask what version should install.
The text was updated successfully, but these errors were encountered: