Skip to content

Conversation

@auouymous
Copy link
Member

Non-media RSS feeds only have a textual description and can't be subscribed to in gpodder. Some media feeds lack downloadable content and can only be accessed from the website. In both cases, gpodder removes the episodes without any feedback to the user as to why, making it appear to be a problem in gpodder.

This patch adds those episodes but prevents automatic download and displays a proper error when a manual download is attempted. The episodes also get a web browser icon to indicate they lack downloadable content. While not downloadable or transferable to media devices, gpodder can be used to notify the user when new episodes are available and they can be accessed by clicking through to the website.

@auouymous auouymous force-pushed the show-episodes-without-enclosures branch from aca63be to 9d29426 Compare April 11, 2022 03:56
@tpikonen
Copy link
Contributor

Seems to work fine, and the code is ok, except for the one point in the review.

Non-media RSS feeds only have a textual description and can't be
subscribed to in gpodder. Some media feeds lack downloadable content and
can only be accessed from the website. In both cases, gpodder removes
the episodes without any feedback to the user as to why, making it
appear to be a problem in gpodder.

This patch adds those episodes but prevents automatic download and
displays a proper error when a manual download is attempted. The
episodes also get a web browser icon to indicate they lack downloadable
content. While not downloadable or transferable to media devices,
gpodder can be used to notify the user when new episodes are available
and they can be accessed by clicking through to the website
@auouymous auouymous force-pushed the show-episodes-without-enclosures branch from 9d29426 to 1cc7941 Compare April 19, 2022 07:24
@elelay
Copy link
Member

elelay commented Apr 24, 2022

LGTM, merge at your leisure

@auouymous auouymous merged commit f490683 into gpodder:master Apr 26, 2022
@auouymous auouymous deleted the show-episodes-without-enclosures branch April 26, 2022 07:13
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.

3 participants