-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[Feature Request]: Publish ESM on CDN #999
Comments
Is this jsDelivr link your wanted feature? |
Since these es files seems to be generated directly from ts files, I'm not sure if they will include some new language features that not work on older (but still supports ESM) browsers, maybe a bundled version can be better |
They are actually bundled with rollup. You can see the bundling configuration here: https://github.com/rrweb-io/rrweb/blob/master/packages/rrweb/rollup.config.js |
I still see some |
I think it's possible but we don't have enough time to create a separate config for this situation. |
I'm going to try to create a PR for this |
I believe this PR might fix it: #1033 |
Preflight Checklist
What package is this feature request for?
rrweb
Problem Description
For pages that asynchronously install rrweb recorder, we currently use
createElement('script')
andscript.onload
to lazy importrrweb-record.min.js
, but it will be more confortable if we can simply useawait import('rrweb-record.esm.js')
.Proposed Solution
Build and publish bundled ESM modules alongside current
dist
Alternatives Considered
Using
createElement('script')
works but is a little complicatedAdditional Information
No response
The text was updated successfully, but these errors were encountered: