-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add a YouTube video renderer component #437
Conversation
Hey @hkirat , Could you please take a look and provide your feedback? |
This needs to be integrated to VideoPlayer.js |
// video_1080p_4: 'https://www.w3schools.com/html/mov_bbb.mp4', | ||
// video_720p_mp4_1: 'https://www.w3schools.com/html/mov_bbb.mp4', | ||
// video_720p_mp4_2: 'https://www.w3schools.com/html/mov_bbb.mp4', | ||
// video_720p_mp4_3: 'https://www.w3schools.com/html/mov_bbb.mp4', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The purpose of this modification was to integrate support for YouTube video URLs within the component. Meanwhile, for the main application, links sourced from the database or other provided sources will remain functional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hkirat, while the linting and formatting for the code pushed to this branch appear to be correct, it seems that the workflow is failing. Could you please investigate and address this issue?
PR Fixes:
Resolves #435
Checklist before requesting a review