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

Treat only really new episodes as new after an update #1297

Merged
merged 3 commits into from
Feb 7, 2023

Conversation

tpikonen
Copy link
Contributor

@tpikonen tpikonen commented Jun 6, 2022

Currently, when a channel or all channels are updated, all episodes marked as new are treated as being new, which usually means being shown in the episode selector. If one treats the 'new' marking as a kind of a to-do list for interesting episodes, sorting through the old, but marked as new episodes after each update gets tiring. This PR makes gPodder treat as new only episodes which are added in an update.

This change could be added behind a new configuration variable, but I decided not to, because episodes marked as new are easily accessible either from View->Unplayed episodes, or from Podcasts->Download new episodes.

@auouymous
Copy link
Member

Would this mean youtube livestreams and premieres require manual downloading when they finally become available? I mark those as new to force a download on the next update, or old if I want to see the duration before downloading.

Marking as old could be used for todo episodes, and deleting when not interested. But I can see how that doesn't work for anyone using the unplayed view mode. Perhaps this could detect the unplayed view setting, or add a custom setting because it might be useful for someone using the other view modes.

@tpikonen
Copy link
Contributor Author

tpikonen commented Jun 8, 2022

Would this mean youtube livestreams and premieres require manual downloading when they finally become available?

Probably. Admittedly this is a use case I did not think about, since I never have auto-download or auto-queue on.

Marking as old could be used for todo episodes, and deleting when not interested.

Well, personally I have (too) many feeds and end up watching or listening only a small fraction of even potentially interesting episodes. Downloading and then deleting would be a waste of bandwidth and storage, and I mostly stream my channels anyway.

So, since there are many ways of using gPodder, should this behavior go behind a config option, with the default being off for backwards compatibility?

@auouymous
Copy link
Member

Have you tested this with the filter extension? For http://borg.qzx.com/test-rss.php, I blocked "aaa" but updating the channel wants to download the newly seen "aaa" episodes.

Downloading and then deleting would be a waste of bandwidth and storage

I use the filter extension to unmark (prevent auto-download) of most channels. By 'delete', I meant deleting the undownloaded episode to get rid of it (using the hide deleted episodes view mode), and either manually downloading, marking new to auto-download on next update, or leaving unmarked if unsure or not ready to download.

should this behavior go behind a config option, with the default being off for backwards compatibility?

That would make it available for all users, no matter what their view mode is. Or it could be a ternary option with off, only if unplayed view, always on. That would allow one who doesn't want it always on to change view mode while updating. Your choice though.

@tpikonen
Copy link
Contributor Author

Rebased to master, added a config variable for this feature and a checkbutton for setting it in the preferences. The filter extension now works as it should.

Also fix the tabbing behavior in preferences by setting 'can-focus' to False in GtkFlowBoxChild containers.

Make PodcastChannel.update() return a list of new episodes. Add a new
config variable ui.gtk.only_added_are_new. If this variable is True,
download, queue or show (depending on the value of config var
auto_download) only these new episodes, instead of all episodes marked
as new, after an update.
@tpikonen
Copy link
Contributor Author

tpikonen commented Jan 6, 2023

Rebased to master. I've been using this for a while now without problems. Could you review this again @auouymous ?

@auouymous auouymous merged commit a3cf8b9 into gpodder:master Feb 7, 2023
@auouymous
Copy link
Member

Merged, thanks for the enhancement, and thanks for fixing the tab issue.

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.

None yet

2 participants