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

Download EPUB files in the background #57

Closed
pmartin opened this issue Oct 20, 2016 · 1 comment
Closed

Download EPUB files in the background #57

pmartin opened this issue Oct 20, 2016 · 1 comment
Labels
Milestone

Comments

@pmartin
Copy link
Owner

@pmartin pmartin commented Oct 20, 2016

For now, EPUB version of entries are downloaded in the foreground, while syncing -- and it works rather fine.

But downloading those files takes time ; and, as the sync is blocking, the user cannot read anything while we are downloading EPUB version of the most recent entries (the ones we are syncing).
It's quite frustrating when no sync has been done for some time (or on first sync ever), as many entries are fetched: downloading EPUB can then take up to a few minutes!

A better way would be to download EPUB version of entries in the background, once the other steps of sync are done. This way, the user could read entries for which EPUB files have already been downloaded before, or the HTML version of entries for which the EPUB version has not been downloaded yet.

@pmartin
Copy link
Owner Author

@pmartin pmartin commented Feb 6, 2017

Downloading in the background is quite complex... And I think it will add too much complexity to the application, for a gain that's not that important.
=> At least for now, I won't do downloads in the background.

Instead, I'll do downloads in the foreground (when syncing), but with several downloads in parallel, to speed things up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.