-
Notifications
You must be signed in to change notification settings - Fork 1.4k
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
overide "downloading on battery" for manual updates #1011
Comments
When Download when not charging is set and you are on battery (not charging), why should the app even ask and not start downloading right away? |
Because I forgot about the preference (again) :-) |
Ups. Added a negation in the post... |
I'm confused, you can still manually click to download whatever episodes you want. The feeds still get refreshed with a manual refresh and show all the episodes to be downloaded in New Episodes. Why not just go there and download the ones you're looking for? Or don't select the Download when not charging option? Alternatively, we could add a 'Download All' option to 'New Episodes'. I'm not sure I like the idea of having it prompt the user. It's certainly not the behavior I'd want. |
The prefs is called If you disagree with me i am fine with closing this issue. I never use the 'new episodes' view or download episodes by hand, as i rely on antennapod to find and download all episodes for me. |
IMHO you are triggering the refresh. The complete refresh then may trigger the automatic download, so the automatic downloads are caused by you rather indirectly. Still, it may be a good idea to have some sort of indicator that auto download is enabled, but is currently not performed because of $reason |
I would suggest a change:
On manual feed update AND episodes are ready to download AND on battery AND "Download when not charging" prefs is NOT set
Antennapod should ask the user:
I tend to forget the reason why the download is not performed, so a manual update should override the prefs or inform me about this situation.
The text was updated successfully, but these errors were encountered: