-
-
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
feat(v7/browser): Add moduleMetadataIntegration lazy loading support #13822
Conversation
I am not 100% sure if it is necessary to backport this to v7 - IMHO this is specialised enough that it appears OK to me to not have this in v7? Especially since |
Currently my platform is in v7 (we initially were in v5 so migrating to v8 is going to be impossible ATM), so I do need it in v7 as well.
inside |
OK, I see, that's fine. Then, instead of having this in
|
42d3e69
to
ec77e5a
Compare
This PR fixes #13803. The corresponding PR for latest major version is #13817.
I saw the bundles generation for integration is happening in
@sentry/integrations
, so I added a corresponding file formodulemetadata
in this package that exports that integration from@sentry/core
, so that the bundle actually gets created for this integration as needed.