Skip to content
This repository has been archived by the owner on Nov 11, 2019. It is now read-only.

nix youtube enclosure in rss feed #291

Open
willkg opened this issue May 4, 2015 · 2 comments
Open

nix youtube enclosure in rss feed #291

willkg opened this issue May 4, 2015 · 2 comments

Comments

@willkg
Copy link
Member

willkg commented May 4, 2015

For videos that are hosted on YouTube, we'd also include the YouTube video url in the enclosure list in the RSS feed because Miro was able to use that and download the video.

Miro is essentially dead, now. I don't know of any other rss clients that work with YouTube video links.

Given that, we should nix the special case enclosure.

@codersquid
Copy link
Contributor

I use NewsBlur as an rss client, and I've seen posts that render with youtube videos. I'll try to dig up an example to see if it is something we can do.

@willkg
Copy link
Member Author

willkg commented May 18, 2015

So I think I want to make the following changes:

  1. nix the youtube-specific code in the RSS enclosures--no more youtube links there
  2. the description of an rss item should include a list of all the urls associated with the item: source url, all media urls, all related urls
  3. (related to rss feeds not being "valid") we should fix the enclosure metadata particularly fileSize, type and length

I want to tackle items 1 and 2 in this issue and tackle item 3 in issue #196 because there's additional data we need to collect in order to do that correctly.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants