-
Notifications
You must be signed in to change notification settings - Fork 67
Added fetch with progress #25
base: master
Are you sure you want to change the base?
Added fetch with progress #25
Conversation
This new implementation of fetch, changes the result data response. But adds functionality to allow progress bar |
Hi @luisgurmendez! Thanks for this PR. |
We could maybe find a way to maintain the same response as fetch, so that the lib is compatible with earlier versions. Let me work on this a few days |
Will this progress bar help in uploading file ? |
@Fausto95 I'm waiting for this feature.. |
Actually it doesn't work because you're not passing the callback function to the S3.uploadFile method! |
Hi, does this allow it possible to get progress on file upload to S3 |
@Fausto95 my bad I forget to push, let me check it |
Thanks buddy, it looks like your branch still not up-to-date with master which now uses TypeScript instead of normal Javascript |
@Fausto95 I've added typescript but it need to be tested I dont' have time right now :S maybe in a week or so |
Any chance this will be merged soon and released? |
Any change this can be merged? |
another vote for this PR |
fyi anyone looking for this feature, i added the PR into a fork here |
No description provided.