[GitHub] [cordova-plugin-file-transfer] wilk-polarny commented on pull request #292: Add throttling to download progress events

[GitBox <gi...@apache.org>]

wilk-polarny commented on PR #292:
URL: https://github.com/apache/cordova-plugin-file-transfer/pull/292#issuecomment-1377342126

   Hi, I would like to suggest a different approach, since fixing the underlying Cordova issue is arduous.
   Rather than always skipping 100 elements, I would suggest to:
   
   a) Only emit computable events
   b) compute and track the progress and only emit an event if the current progress changed significantly. For example, computer the progress %, and only emit an event every 1% - or once a certain amount of transferred bytes has been reached - based on the content length.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org