Skip to content

Make podcasts work sanely.#387

Open
hrdwrbob wants to merge 5 commits intofondberg:masterfrom
hrdwrbob:master
Open

Make podcasts work sanely.#387
hrdwrbob wants to merge 5 commits intofondberg:masterfrom
hrdwrbob:master

Conversation

@hrdwrbob
Copy link

This PR changes podcasts to work as any user would expect - it plays the podcast (not an episode - which is what it was doing), and it plays it from the first episode that hasn't been fully played, resumed at the point it was.

The previous way it worked would only ever play a single episode and then stop, and it didn't work in ways any user would reasonably expect - I have therefore also removed the ignore_fully_played option. I have added the position == 1 overload, which makes it search backwards, playing the latest unplayed episode first.

The main issue here is that you need to go through all the episodes and count the number yourself - you can't play a show with the uri of an episode. It works, but it's terrible.

@fcusson
Copy link
Collaborator

fcusson commented Mar 19, 2023

Hi @hrdwrbob, we had a problem in the structure of the manifest file (it seems that Home Assistant decided that all entries must be alphabetically sorted from now on). That is the reason your validation failed. I updated the file, but you will need to resolve your merge conflict in order to be pushed into the code base.

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