uintdev / qrserv

Transfer files with ease over a network.
https://play.google.com/store/apps/details?id=dev.uint.qrserv
MIT License
135 stars 9 forks source link

Time taking to create server #13

Closed brnwlshubh closed 8 months ago

brnwlshubh commented 8 months ago

Why it is taking too much time?

uintdev commented 8 months ago

Information on that can be found under PHILOSOPHY.md.

In short, due to the app making use of Flutter, there were limited existing resources in terms of file selection and so it all went down to copying the entire file selection into the app's cache. The copying process takes time and especially for huge files. This also introduces other issues, such as temporary increased storage usage.

Ideally, we need a way to select files from storage but only get a full path to those files. I might be eyeing something that might be able to achieve this, but I'll have to test to make sure features (such as multi-file selection) won't be lost and it works as intended. (Even if achieved, there doesn't appear to be a way for share sheets, so that would remain an issue. Should be fine with several photos. Large videos? Maybe a little more of a struggle.)

uintdev commented 8 months ago

Small update: I had put together a basic test, and it's beginning to look promising. Will need to begin implementing into the app itself to be more certain if it would work. As there's less control over the file manager that would be presented (including the theme and i18n) and it does not offer multi-selection, if all goes well, this will just be an optional way to import files (for those huge ones). The usual downloads/documents app is convenient most of the time and more user friendly, in any case.

Slight concern, though. When it comes to the Play Store, Google is a bit strict when it comes to the usage of MANAGE_EXTERNAL_STORAGE. It would have perfectly legitimate use in this case, so I hope that they also agree on that.

uintdev commented 8 months ago

Version 2.2.0 has been released with a ‘direct access mode’ that should help avoid such issue with large files.

uintdev commented 8 months ago

Closing as resolved.