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

Feature playlists #35

Open
wants to merge 2 commits into
base: playlists-feature
Choose a base branch
from

Conversation

drhoet
Copy link

@drhoet drhoet commented Dec 28, 2015

Adding simple support for youtube playlist URLs

Remark: same as #34 but now on playlists-feature branch as requested.

@dirkjanm
Copy link
Owner

Thanks, I'll have a look at merging it in, am I correct in assuming that when a video is part of a playlist it always sends the whole playlist to Kodi?

@drhoet
Copy link
Author

drhoet commented Dec 28, 2015

Yes it does. The kodi plugin already supported playlists and loads the whole thing when it receives a playlist id. Might be cleaner if the user gets the option to send the playlist or only the video (e.g. different link in the context menu)? I didn't search for it further because I didn't really need it, but I can look into it if you want...

@dirkjanm
Copy link
Owner

Its all right for now, there are several things that need solving in the playlists branch, I will have a look on how to implement it.

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.

2 participants