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 Youtube/Vimeo/... videos in the ZIM #13

Open
benoit74 opened this issue Oct 1, 2024 · 6 comments
Open

Add Youtube/Vimeo/... videos in the ZIM #13

benoit74 opened this issue Oct 1, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request question Further information is requested
Milestone

Comments

@benoit74
Copy link
Contributor

benoit74 commented Oct 1, 2024

Libretexts rely on Youtube /Vimeo/... videos at some places, either "inline" with an <iframe> or "external" with an <a> external link which opens the video in another tab / window.

Sample pages:

We need to :

  • download, reencode, cache on S3 these videos
  • add these reencoded videos to the ZIM
  • fix HTML code to use video.JS plugin to display these reencoded videos
    • in-place when an <iframe> was used
    • in a popup when an <a> was used

But first (@kelson42 @Popolechien @rgaudin ):

  • do we really want / need to include all these videos in the ZIM? (I suspect this can make the ZIM quite big)
  • do we want to build different flavors? (no video at all, only inline videos, all videos?)
@benoit74 benoit74 added enhancement New feature or request question Further information is requested labels Oct 1, 2024
@benoit74 benoit74 added this to the 0.1 milestone Oct 1, 2024
@benoit74 benoit74 self-assigned this Oct 1, 2024
@benoit74
Copy link
Contributor Author

benoit74 commented Oct 1, 2024

Other question: is this something required for libretexts project? I'm quite sure we never discussed this, so maybe this is simply out of scope for now and we should just drop iframe and replace them with an external link.

@Popolechien
Copy link

Flavours: no.
Including them: well yes, I assume they're part of the teaching material. Also Vimeo has been nagging us for a while, might as well seize the occasion.

@benoit74
Copy link
Contributor Author

benoit74 commented Oct 1, 2024

Including them: well yes, I assume they're part of the teaching material.

I had a look at only few videos and they are more "resources" for the ones who want to dig a bit rather than essential learning material. From my PoV, they are part of the teaching material just like all other external links, which we do not include. So your argument seems a bit weak in my PoV.

Also Vimeo has been nagging us for a while, might as well seize the occasion.

Unfortunately work on supporting Vimeo here will not help supporting Vimeo in zimit, if this is what you are referring to.

@Popolechien
Copy link

Ok let me ask the client just to confirm.

@Popolechien
Copy link

Ok here is their answer:

Scraping videos is nice, but there are clear intellectual property issues here. Youtube may host Creative Commons openly licensed videos or may use a YouTube License (https://www.termsfeed.com/blog/youtube-license-types/). The latter doesn't allow for reuse permissions that scrapping represents. So, only Creative Commons videos can be legally scrapped and distributed via your infrastructure.
So, to scrap a video, you would have to use the YouTube API to query its license (I presume there is an endpoint for this) before scraping. If it is a Creative Commons license, I am game with scraping. If YouTube license, then the best we can do is substitute a hyperlink to the video (or keep it as a hyperlink).

Not sure we can ask the Youtube API for the license, so the answer at this stage is "no videos"

@benoit74
Copy link
Contributor Author

benoit74 commented Oct 3, 2024

OK, thank you. I will postpone this issue to "later" since we might never implement this. I've opened a dedicated issue for the addition of videos in the ZIM.

@benoit74 benoit74 modified the milestones: 0.1, later Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants