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

spouts/youtube: Various improvements #1412

Merged
merged 6 commits into from
Feb 16, 2023
Merged

spouts/youtube: Various improvements #1412

merged 6 commits into from
Feb 16, 2023

Conversation

jtojnar
Copy link
Member

@jtojnar jtojnar commented Feb 15, 2023

  • Support handles

@netlify
Copy link

netlify bot commented Feb 15, 2023

Deploy Preview for selfoss canceled.

Name Link
🔨 Latest commit 9e74bcc
🔍 Latest deploy log https://app.netlify.com/sites/selfoss/deploys/63ed7d488fb2800008e5890d

@jtojnar jtojnar added this to the 2.20 milestone Feb 15, 2023
@jtojnar jtojnar force-pushed the youtube-improvements branch 3 times, most recently from ef18fb7 to a0848f0 Compare February 16, 2023 00:38
Handles such as @BreakingTaps or @MinuteFood will be appended to https://www.youtube.com/ and we will let feed discovery obtain the feed URL.

https://www.youtube.com/handle
https://support.google.com/youtube/answer/6180214?ref_topic=9257109

Fixes: #1389
The item content is always empty. There is only textual description in the enclosure, which we are not extracting yet.
The link will just contain the URL for use in frames so it is useless as thumbnails.
It would replace apostrophes with `'` breaking the content equality assertion in the next commit.
This will allow for better filtering.
@jtojnar jtojnar force-pushed the youtube-improvements branch from a0848f0 to 9e74bcc Compare February 16, 2023 00:48
@jtojnar jtojnar merged commit 9e74bcc into master Feb 16, 2023
@jtojnar jtojnar deleted the youtube-improvements branch February 16, 2023 00:49
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.

1 participant