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

Feature/wifi only downloads #60

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

BenOien
Copy link

@BenOien BenOien commented Mar 23, 2016

Implemented WiFi-only downloads:

Behavior:
*If WiFi is unavailable, WiFi-only downloads are queued.
*When Wifi connects, WiFi-only downloads are started.
*If WiFi is disabled, it is up to the consuming app or user to enable WiFi.

Api:
*DownloadRequest has a new method for indicating WiFi only: setWifiOnly(boolean, Context).
*This method includes a context to maintain full api backward compatibility.
*Using this requires additional permissions: see permissions section in README. No additional permissions are required until after a WiFi-only DownloadRequest has been added.

Fix:
*When a new download is added to the queue, cancel active downloads with the same destination file.

TestApp:
*Dynamically created requests: serves to reset DownloadRequest status on repeated requests.
*Added newly required permissions for WiFi only.

Thank you for your time and consideration.

Benjamin Oien and others added 3 commits March 23, 2016 13:42
*TestApp: Dynamically created requests: serves to reset download status on repeated requests. Added newly required permissions.
…oadQueue

*Better setWifiOnly JavaDoc comment
*Test App: Easier to tell if a download request was wifi-only. Only first download is wifi-only
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.

1 participant