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

auto enqueue episodes #133

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@boyska

boyska commented Jan 14, 2016

(basically copied from enqueue_in_mediaplayer.py)

This allows the user to automatically enqueue episodes while they're being downloaded.

auto enqueue episodes
basically copied from enqueue_in_mediaplayer.py
@thp

This comment has been minimized.

Show comment
Hide comment
@thp

thp Jan 15, 2016

Member

Instead of creating a completely new extension and duplicating lots of code, maybe you can add a configuration option to the original extension that would make the enqueueing after download opt-in?

Member

thp commented Jan 15, 2016

Instead of creating a completely new extension and duplicating lots of code, maybe you can add a configuration option to the original extension that would make the enqueueing after download opt-in?

@thp thp closed this in ef76d03 Jan 15, 2016

@thp

This comment has been minimized.

Show comment
Hide comment
@thp

thp Jan 15, 2016

Member

Merged it into the original extension now, thanks for the suggestion!

Member

thp commented Jan 15, 2016

Merged it into the original extension now, thanks for the suggestion!

@boyska

This comment has been minimized.

Show comment
Hide comment
@boyska

boyska Jan 15, 2016

But isn't it counter intuitive that it is not in the post-download category? As a user, how do I understand that there is such a functionality?

boyska commented Jan 15, 2016

But isn't it counter intuitive that it is not in the post-download category? As a user, how do I understand that there is such a functionality?

@thp

This comment has been minimized.

Show comment
Hide comment
@thp

thp Jan 15, 2016

Member

True, we should probably change its name and put it into two categories or something.

Member

thp commented Jan 15, 2016

True, we should probably change its name and put it into two categories or something.

@boyska

This comment has been minimized.

Show comment
Hide comment
@boyska

boyska Jan 15, 2016

Is it possible? I understood that a single extension can be in just one category, which is why I created a different one.
I don't know if it's possible to share code between extensions.

boyska commented Jan 15, 2016

Is it possible? I understood that a single extension can be in just one category, which is why I created a different one.
I don't know if it's possible to share code between extensions.

@thp

This comment has been minimized.

Show comment
Hide comment
@thp

thp Jan 16, 2016

Member

Well not yet, we'd have to change the code that parses the categories to also make it available there. But that shouldn't be too hard to do?

Member

thp commented Jan 16, 2016

Well not yet, we'd have to change the code that parses the categories to also make it available there. But that shouldn't be too hard to do?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment