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

Make episodes not in inbox eligible for auto-download #6671

Open
3 tasks done
Husaberger opened this issue Oct 1, 2023 · 36 comments
Open
3 tasks done

Make episodes not in inbox eligible for auto-download #6671

Husaberger opened this issue Oct 1, 2023 · 36 comments
Labels
Needs: Decision Proposal and most arguments are clear, but needs a verdict. Type: Feature request

Comments

@Husaberger
Copy link

Checklist

  • I have used the search function for open and closed issues to see if someone else has already submitted the same bug report.
  • I will describe the problem with as much detail as possible.
  • If the bug only to occurs with a certain podcast, I will include the URL of that podcast.

App version

3.1.1

Where did you get the app from

Google Play

Android version

13

Device model

Pixel 7

First occurred

3 months ago

Steps to reproduce

All podcasts only download manually despite uninstall/reinstall. All auto download settings unchanged from when AP did auto downloaded

Expected behaviour

Auto download

Current behaviour

Not auto download

Logs

No response

@Husaberger Husaberger added the Type: Possible bug Issues that seem to be a bug, but haven't been confirmed yet label Oct 1, 2023
@Husaberger
Copy link
Author

Environment

Android version: 13
OS version: 5.10.157-android13-4-00003-g776d0a76f6aa-ab10208116
AntennaPod version: 3.1.1
Model: Pixel 7
Device: panther
Product: panther

No crash report recorded

@ByteHamster
Copy link
Member

Do you use a VPN? Try turning it off.

@Husaberger
Copy link
Author

No VPN is bring used

@ByteHamster
Copy link
Member

WiFi or mobile network? "Automatic download when on battery" enabled? Is there a little icon next to the downloads section on the navigation bar?

@Husaberger
Copy link
Author

Both WiFi and mobile
Download when not charging currently off- but also fails to auto download when toggled on
Do not see download icon even when manually downloading

@maniyasujit
Copy link

maniyasujit commented Oct 3, 2023

Yes, I faced the same issue yesterday while I was downloading the podcast. (I have a debug version of the application that I have installed using Android Studio).

@Husaberger
Copy link
Author

they must have changed something because now all podcasts are auto downloading again

@JasonMaggini
Copy link

I'm having this issue as well. Automatic downloads just don't work. Battery settings, charging, not charging, wifi, mobile, no combination of anything has worked for me. I've changed episode cache up and down, I've wiped all the data, reinstalled, tried only one podcast subscription, nothing.

@Husaberger
Copy link
Author

Most of my podcasts now download automatically. I only have 2 that do not. Would like to know if AP changed something to make most of them download automatically

@ByteHamster
Copy link
Member

No, we did not change anything. There was no update and we do not have a server that we could change.

@Husaberger
Copy link
Author

Husaberger commented Oct 10, 2023 via email

@tonytamsf
Copy link
Member

for people who have problems, go to Downloads and click on history. what you are describing is a problem on the server side that AntennaPod does not control.

here is a screenshot showing problems specific to the feeds I am subscribed to

Screenshot_20231010-105348.png

@JasonMaggini
Copy link

Download log is blank on mine, as the app will never attempt to automatically download, regardless of settings.

@JasonMaggini
Copy link

After some experimenting, it seems like 3.0.2 is the last version that will reliably autoupdate for me.

@Husaberger
Copy link
Author

for people who have problems, go to Downloads and click on history. what you are describing is a problem on the server side that AntennaPod does not control.

here is a screenshot showing problems specific to the feeds I am subscribed to

Screenshot_20231010-105348.png

Helpful
Did not know that log existed

@rahqbo
Copy link

rahqbo commented Nov 30, 2023

I too have this problem. I updated from Arrowos Android 12.1 to Crdroid Android 13 and installed Antennapod again, fresh. Auto download active, also when not charging. Podcasts are coming in, but not getting downloaded. The Antennapod version has not changed, both the latest from F-Droid.

@keunes
Copy link
Member

keunes commented Dec 2, 2023

Download log is blank on mine, as the app will never attempt to automatically download, regardless of settings.

@JasonMaggini The log should also display items that you manually downloaded. Did you try that?

After some experimenting, it seems like 3.0.2 is the last version that will reliably autoupdate for me.

Here you can see the code changes that landed after 3.0.2 until & including 3.1.1 (which is the version mentioned in the first post):

3.0.2...3.1.1

Maybe you can check if there are any changes that might point to anything related to (auto)downloads?

@keunes
Copy link
Member

keunes commented Dec 2, 2023

Just a thought @JasonMaggini; is your auto refresh enabled? Without refresh no auto-download.

@damoasda
Copy link
Contributor

damoasda commented Dec 3, 2023

I had the same issue. After I switched the setting "New episodes action" back to "Add to inbox", it worked again for me.

This behaviour is caused by the method DBTasks.updateFeed():

if (action == FeedPreferences.NewEpisodesAction.ADD_TO_INBOX
        && (item.getPubDate() == null
            || priorMostRecentDate == null
            || priorMostRecentDate.before(item.getPubDate())
            || priorMostRecentDate.equals(item.getPubDate()))) {
    Log.d(TAG, "Marking item published on " + item.getPubDate()
            + " new, prior most recent date = " + priorMostRecentDate);
    item.setNew();
}

This seems to be a feature, implemented in February this year, see #6442.

@rahqbo
Copy link

rahqbo commented Dec 12, 2023

I had the same issue. After I switched the setting "New episodes action" back to "Add to inbox", it worked again for me.

This behaviour is caused by the method DBTasks.updateFeed():

if (action == FeedPreferences.NewEpisodesAction.ADD_TO_INBOX
        && (item.getPubDate() == null
            || priorMostRecentDate == null
            || priorMostRecentDate.before(item.getPubDate())
            || priorMostRecentDate.equals(item.getPubDate()))) {
    Log.d(TAG, "Marking item published on " + item.getPubDate()
            + " new, prior most recent date = " + priorMostRecentDate);
    item.setNew();
}

This seems to be a feature, implemented in February this year, see #6442.

I can confirm your findings.

@keunes
Copy link
Member

keunes commented Dec 13, 2023

Ping @terminalmage. Would you have any idea?

@ByteHamster
Copy link
Member

This is intended. Only episodes in the inbox are eligible for auto-download.

@keunes
Copy link
Member

keunes commented Dec 13, 2023

I think we need to re-discuss. If the user wants to skip the inbox, it can be because they don't want it (for which there are other methods) or because they want all of them.

@keunes keunes added Needs: Decision Proposal and most arguments are clear, but needs a verdict. Type: Enhancement and removed Type: Possible bug Issues that seem to be a bug, but haven't been confirmed yet labels Dec 13, 2023
@ByteHamster
Copy link
Member

If you want all of them, enabling the inbox makes no difference because they are removed again once downloaded

@keunes
Copy link
Member

keunes commented Dec 13, 2023

If you want all of them, enabling the inbox makes no difference because they are removed again once downloaded

Exactly. So why, if all episodes of a given podcast should be downloaded anyway, do we make them appear briefly in the Inbox without purpose.

@ByteHamster
Copy link
Member

If you hit your auto-download limit or if all retries fail, it remains in the inbox

@rahqbo
Copy link

rahqbo commented Dec 13, 2023

I propose some kind of hint for this behaviour in the settings. Something like "Disabling the inbox disables auto-downloads, too." or "The inbox is required for auto-downloads to work." or "Only podcasts appearing in the inbox are elegible for auto-download." Or grey out the auto-download settings when the inbox is disabled.

@LRitzdorf
Copy link

Just my two cents, as a recent switch-over from Google Podcasts: I encountered this issue because I use my push notifications as something of an inbox. I just don't see the utility of the in-app inbox for my personal workflow.

Before finding this issue, I wouldn't have expected the current behavior at all — this may just be me, but I inferred the "new episode" flow to be "notify (inbox, push, or maybe both), then optionally download." Accordingly, I had disabled the inbox, then enabled push notifications and auto-download.

The fact that downloaded episodes are removed from the inbox helps, but that means I currently need to touch two settings per podcast to actually allow downloads: enabling the inbox, and then enabling auto-download.

Related question: is the inbox also required for push notifications to work? I found this thread after an episode failed to download (because no inbox), and I don't recall having seen a push notification for that episode, either. (That one episode is the only one to have been released since I switched to AntennaPod, so my testing is a bit limited, sorry.)

@keunes
Copy link
Member

keunes commented Dec 24, 2023

is the inbox also required for push notifications to work?

As far as I'm aware, it is not. However, episode notifications are not enabled by default, and must be activated at the level of podcasts.

@terminalmage
Copy link
Contributor

When I added the option to skip the inbox, I misunderstood what @ByteHamster mentioned here. Like @rahqbo, the inbox just doesn't make sense for me. In fact, it actually makes the user experience worse. I subscribe to several feeds which include "full show" and hourly episodes in the same feed. I don't want to listen to all of them, obviously, because there is a lot of duplication. So I do the following:

  1. Configure auto-download filters for that feed to exclude the hourly episodes
  2. Mark the hourly episodes as played

However, this leaves the undownloaded episodes in the inbox. So when I use the swipe action to change played state, I have to swipe once to remove from the inbox, and a second time to mark as played.

@ByteHamster
Copy link
Member

@terminalmage Keunes and I had a 3-hour meeting yesterday and discussed how to deal with this. We decided to do a rewrite of the auto-download feature, so that it becomes independent from the inbox.

@terminalmage
Copy link
Contributor

Oh, nice! That would definitely help.

@keunes
Copy link
Member

keunes commented Dec 30, 2023

Currently writing up the forum post with full notes of our meeting. Will share the link here once it's ready.

I subscribe to several feeds which include "full show" and hourly episodes in the same feed. I don't want to listen to all of them, obviously, because there is a lot of duplication. So I do the following:

  • Configure auto-download filters for that feed to exclude the hourly episodes
  • Mark the hourly episodes as played

@terminalmage I guess that, alternative to having the 'new episode action' setting, the combination of #6054 + #5237 would cover you as well.

@ByteHamster
Copy link
Member

I guess that, alternative to having the 'new episode action' setting

Wait, so now we are considering removing the setting again?

@keunes
Copy link
Member

keunes commented Dec 30, 2023

Wait, so now we are considering removing the setting again?

You said it first! 😉

I would keep it, because I think it's still valid/useful for other use cases.

@antennapod-bot
Copy link

This issue has been mentioned on AntennaPod Forum. There might be relevant details there:

https://forum.antennapod.org/t/new-episode-action-enhancement/3871/1

@AntennaPod AntennaPod deleted a comment from antennapod-bot Jan 4, 2024
@ByteHamster ByteHamster changed the title Auto download not working Make episodes not in inbox eligible for auto-download Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Decision Proposal and most arguments are clear, but needs a verdict. Type: Feature request
Projects
None yet
Development

No branches or pull requests