-
Notifications
You must be signed in to change notification settings - Fork 129
fix: VideoJS 8 support #396
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
base: main
Are you sure you want to change the base?
Conversation
@gkatsev , @philjhale Let me know if something else needs to be changed. |
@alex-barstow Can you please take a look on this? |
Any update on this PR on when this will be merged, as I am having the same issue with videoJS8? |
This isn't sufficient yet for Video.js 8 compatibility. The same issue mentioned before will need to be addressed. The build needs to not transpire ES6 classes. This will likely need updates to several dependencies. |
Okay @mister-ben . Let me try by updating |
@mister-ben Can you please take a look? Let me know if any further changes are needed. |
@gkatsev I am having the same problem with VideoJS-8. |
Description
Allow videojs 8
Specific Changes proposed
rollup
andvideojs-generate-rollup-config
versions.Requirements Checklist