Skip to content
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

Dev2.5.18 generate external_videos.xml for playing on the fly #215

Open
wants to merge 11 commits into
base: test2.5.18_showDiff_extVideoXml
Choose a base branch
from

Conversation

hiroshisuga
Copy link
Owner

@hiroshisuga hiroshisuga commented Jun 18, 2023

What does this PR do?

Generate external_videos.xml for play it back on the fly.
See: bigbluebutton/bbb-playback#127
You need to directly modify the script /usr/local/bigbluebutton/core/lib/recordandplayback/generators/events.rb and
/usr/local/bigbluebutton/core/scripts/publish/presentation.rb and
/usr/local/bigbluebutton/core/scripts/presentation.yml

Closes Issue(s)

Closes #

Motivation

More

  • Added/updated documentation

@hiroshisuga hiroshisuga changed the title Dev2.5.18 ext video xml Dev2.5.18 generate external_videos.xml for playing on the fly Jun 18, 2023
add:
include_external_videos: true
which will be used in publish/presentation.rb
generate shapes.svg and presentation/externalVideo.png for thumbnails of external videos
@sonarcloud
Copy link

sonarcloud bot commented Jun 23, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

stupid mistake
Condition to record the external video playing fixed.
Revert one condition, which however seems unnecessary..
Omit 'round' for this.player.getCurrentTime() so that the player time becomes float.
remove .to_i for making the timestamp float in the external_video.xml file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant