-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Improve modulesIntegration
for ESM
#14427
Comments
hey @lforst, i was thinking about working on this, so i started to look into the code for |
@dumbbellcode Yup that's totally possible. Historically we have neglected this integration a bit because it doesn't provide too much value to Sentry users directly. However, it can be immensely helpful to us when we help troubleshoot issues with the SDK. |
which means there are 2 distinct issues to work on,
I am planning to work on both but I think it's important to fix the 1st issue itself as it makes the integration obsolete if not fixed, could you create an issue and assign it to me ?, i'll give it a shot. Thanks ! ping: @lforst |
@dumbbellcode feel free to just submit a PR outlining the problem and how you're solving it. Thanks! |
sure @lforst, will create an issue and submit a PR in some time 👍 |
Description
Currently the
modulesIntegration
does not work for ESM. Maybe we can use import-in-the-middle to improve that situation.The text was updated successfully, but these errors were encountered: