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

Add Echo360 EchoVideos as a provider #182

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

echo360-damyon
Copy link

At Echo360 (https://echo360.com/) we have been asked by our customers to allow videos captured in our EchoVideo platform to be able to be included in H5P content. We have added these features in our test environment and they will be released generally at the end of the year.

For this to work in the same way as other supported video platforms it will require changes to both this repository as well as the H5P Video library. Our changes are modelled after the other video platforms.

},
{
name: 'Echo360',
regexp: /^[^\/]+:\/\/(echo360[^\/]+)\/media\/([^\/]+)\/h5p.*$/i,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a note: If Echo360 requires to have an extra link to serve videos for H5P in addition to the public link that one gets when intending to share the video, and that's not too confusing for Echo360 users, that regex seems fine.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks - yes Echo360 requires a separate route which triggers some minor layout changes and feature changes

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.

None yet

2 participants