stonith404
feat(Share): Generate zip file on the fly
Goal of this PR is to no longer generate zip files for all shares by default but rather generate them on the fly when a download is requested. The only reason for this to generate overhead over someone downloading the files one by one would be the compression - which one could get rid of.
Not completely finished yet, need to remove the zip generation and saving on upload and the isZipReady property. I have tested this already though, the API now generates the zip file when requested.
Would this also apply for the S3 storage option? As currently the S3 storage doesn't have the ZIP functionality.
I would assume that this should also work with S3. However I never used S3 and don't intend to, so I won't be able to verify.
- I now always show the
Download Allbutton, even if there is just one file. That makes the UI consistent and because the ZIP can now also be generated for single files I don't see a reason not to do it. This is personal preference however, I can undo this change again. - The ZIP is now generated every time it is downloaded meaning the compression is also done every time. Personally I'd just remove the compression setting (not yet done though), with compression level 0 the ZIP generation only has negligible overhead compared to just downloading each file.
@stonith404 can you give me some input on this?
Is it enough to remove the translation keys for english and Crowdin will remove it for the others as well or do you have to remove them all manually?
I would assume that this should also work with S3. However I never used S3 and don't intend to, so I won't be able to verify.
There is probably a way but I'm not using S3 as well so someone else would have to contribute to support S3.
I now always show the Download All button, even if there is just one file. That makes the UI consistent and because the ZIP can now also be generated for single files I don't see a reason not to do it. This is personal preference however, I can undo this change again.
Yeah that makes sense.
The ZIP is now generated every time it is downloaded meaning the compression is also done every time. Personally I'd just remove the compression setting (not yet done though), with compression level 0 the ZIP generation only has negligible overhead compared to just downloading each file.
Yes we can do that. If anyone misses the compression feature, they can submit a feature request, and we can consider reintroducing it.
Is it enough to remove the translation keys for english and Crowdin will remove it for the others as well or do you have to remove them all manually?
Crowdin will remove the strings from the other files.
Hello, I am very interested in this PR as I am using S3 and for now when we click on the "Download all" button, we get a "Bad Request" error.
Is there anything I can help with to implement this to the S3 service in this PR ?
Thanks !
