Skip to content

Support for OttFlix videos in H5P#155

Open
eduardokraus wants to merge 2 commits intoh5p:masterfrom
EduardoKrausME:master
Open

Support for OttFlix videos in H5P#155
eduardokraus wants to merge 2 commits intoh5p:masterfrom
EduardoKrausME:master

Conversation

@eduardokraus
Copy link

This feature adds compatibility with the OttFlix platform within H5P, allowing educators and content creators to use videos hosted on OttFlix directly in their interactive activities.

@otacke
Copy link
Contributor

otacke commented Aug 19, 2025

Disclaimer: I am not part of H5P Group, so feel free to conpletely ignore my suggestions (which are based on a quick glance only anyway and not exhaustive).

  1. You should ensure that your code adheres to the H5P Coding styleguide, e. g. indentation, positioning of else, use of == instead using ===, ...
  2. While old H5P code uses ES5, newly written code should use ES6.
  3. Are you sure this works without also modifying the list of providers in the AV widget?
  4. H5P Group needs to test this, so you should supply information on how they can get hold of an OttFlix instance to do so.
  5. You might want to check if this works properly for both, standalone videos and videos used in InteractiveVideo. This has caused issued here and there for other video handlers before.

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.

2 participants