Closed andre-huehn closed 6 years ago
I also added the option to auto delete the file after succesful upload
test and test2 were commits to install from my public fork, i can always undo them later on.
Hi and thanks for your work! I have a couple of suggestions that should be addressed before the PR can be merged.
READ_EXTERNAL_STORAGE
permission in nativescript-background-http/platforms/android/AndroidManifest.xml
should be replaced with WRITE_EXTERNAL_STORAGE
(which includes read and write):
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
package.json
files should not be modified for this request as it does not contain updated dependencies. You can undo those so only .ts
and .d.ts
files are modified.Done undoing the package.json changes and added the WRITE permissions. Looking forward for the next release. :)
Implemented canceling running uploads
I needed some trickery to get tsc to compile the code. (some casts)
I also change the ios implementation to return common.task instead task for that.
Please review my changes.