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

load youtube channels from www.youtube.com/{key} #306

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

biolds
Copy link
Contributor

@biolds biolds commented Jan 17, 2023

Some channel return a 404 with the /c/ prefix
(ie. https://www.youtube.com/c/@relaxingmusicandvibes)

@biolds biolds mentioned this pull request Jan 17, 2023
@biolds
Copy link
Contributor Author

biolds commented Jan 17, 2023

Follow up on #305 (comment), how should these kind of channels be supported ? I didn't know there were different kind of channels..

@meeb
Copy link
Owner

meeb commented Jan 17, 2023

That's because they're not channels, they're user pages or other aliases. The only real way to make a sane interface for people (probably) is to just accept any URL, pull the metadata via yt-dlp and then check for a channel ID, then suggest using that. For example, the actual channel name for @relaxingmusicandvibes is https://www.youtube.com/channel/UCxbIu0sL2NwqzbeU65CxAbw which isn't exactly obvious. Edit: I have a fork I was playing with that does attempt to auto-lookup channel IDs from vanity user pages, however currently it's too slow and overly messy so it's not merged yet.

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.

None yet

2 participants