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

[feed] added channel for enclosure link on latest entry #11257

Closed
wants to merge 8 commits into from
Closed

[feed] added channel for enclosure link on latest entry #11257

wants to merge 8 commits into from

Conversation

juergenpabel
Copy link
Contributor

[feed] added channel for enclosure link on latest entry

The feed plugin doesn't support (most) podcasts, as the audio link is provided in an enclosure tag; this PR adds the new channel "lastest-enclosure" to the thing. Albeit multiple enclosures are supported, this thing exposes only the first enclosure (if any at all - else the channel will be UNDEF). This covers the vast majority of podcasts sufficiently while minimizing the changes to the binding (a general re-refactoring/improvement of this binding would seem to be reasonable, though).

@Skinah Skinah added the enhancement An enhancement or new feature for an existing add-on label Sep 30, 2021
@lolodomo
Copy link
Contributor

lolodomo commented Oct 2, 2021

Please add your new channel in the documentation and please sign-off your PR.
Except that, this is good for me.
@svilenvul : can you please have a look ?

juergenpabel and others added 5 commits October 4, 2021 12:44
Signed-off-by: Juergen Pabel <juergen@pabel.net>
Signed-off-by: Kai Kreuzer <kai@openhab.org>
Signed-off-by: Juergen Pabel <juergen@pabel.net>
Signed-off-by: Juergen Pabel <juergen@pabel.net>
Signed-off-by: Juergen Pabel <juergen@pabel.net>
@juergenpabel
Copy link
Contributor Author

Gnahh....included commit https://github.com/juergenpabel/openhab-addons/commit/195d532aa770aa69e78d103322fabe4a0295f96f from Kai in rebase signoff. Please ignore (wouldn't be merged anyhow)

@juergenpabel
Copy link
Contributor Author

Closing this PR in favor of (#11345) for cleanliness sake.

@juergenpabel juergenpabel deleted the feed_enclosure-link branch October 4, 2021 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants