search

h5p / h5p-video

10 stars 73 forks source link

Nanoo.tv video support #51

Open tobiasreischmann opened 3 years ago

tobiasreischmann commented 3 years ago

This patch provides an additional video-player, which enables to use h5p for videos provided by the swiss educational platform nanoo.tv.

It would be really nice, if this could be integrated. Please, let me know, if any changes to the code are necessary, in order to make the integration work. Cheers, Tobias

atrottmann commented 3 years ago

I am working at Werft22, the company behind nanoo.tv. We are willing to continue supporting this integration, as many of our customers want to use nanoo.tv videos with H5P's "interactive video" feature.

tkorner commented 3 years ago

I'm using this plugin as a beta tester already and I am very happy with it. I appreciate if you merge this commit into the code.

disclaimer: I'm using this feature in my high school but work as an external advisor for Werft22 too.

atrottmann commented 3 years ago

Is there anything we can do to help getting this merged?

atrottmann commented 3 years ago

Thank you very much; I'll check back with @tobiasreischmann so we can do this as soon as possible.

@fnoks: If/when you want to test it, you may use the following link to a video: https://www.nanoo.tv/link/v/Moodle

atrottmann commented 3 years ago

I think the requested changes have now been committed - @fnoks, can you please check again?

atrottmann commented 3 years ago

@fnoks, is there anything we can do to resolve the changes requested?

Kind rgards

Andreas Trottmann

StefanRickli commented 3 years ago

Any updates? I would be very happy to see this integrated.

tzulli commented 3 years ago

I am writing on behalf of all teachers who would like to use H5P with nanoo.tv. Could you please complete the code as described above.

AnitaNixon commented 3 years ago

Hi there, we have currently around 40'000 users, mainly teachers, that use our video streaming and online-library tool nanoo.tv More and more customers ask us for H5P.

Is there anything you need from our side in order to make this integration happen? We are planning a campaign in october and would be happy to offer this often demanden feature to our customers.

tkorner commented 3 years ago

just to clear the field: nanoo.tv has its user base in Switzerland only, but for that it is the market leader (especially in K12, which is one of the target audience of h5p too). Merge is be very appreciated.

EricGuic commented 1 year ago

Hi, I have a similar needs that this one, but I see that this PR hasn't been merged for a while now. Is that because the Change requested hasn't been made by @tobiasreischmann or is it for another reason ? Thanks for the update before I create a similar PR for another provider (Peertube). Best regards.

graphics80 commented 11 months ago

@fnoks We (https://dlh.zh.ch/) would also support this PR as nanoo.tv is the market leader in Switzerland and is widely used in education.

It seems all the changes requested:

Thank you for contributing to H5P, and sorry for this taking so long...

Before I test this PR, I would like to see the following changes:

  • document.getElementById(id) shouldn't be neccessary, since the reference to the iframe is created by this code (i.e: player)
  • contentWindow.postMessage is duplicated many places. Could you please wrap this inside a function?

have been implemented. Is there anything we can do to speed this up?