-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Estimate properly or not show ETA for uploaded files using the web UI. #25053
Comments
FYI @ckamm |
Or at least remove the message "Any time soon..." which is likely to appear when the upload is short. |
This has changed from 9.0 to 9.1 |
Maybe the algorithm calculating the remaining time can be improved, so there is less jumping in the values. |
Hey, this issue has been closed because the label (This is an automated comment from GitMate.io. |
Hey, this issue has been closed because the label (This is an automated comment from GitMate.io.) |
Hey, this issue has been closed because the label (This is an automated comment from GitMate.io.) |
Moving to planned. We should tackle this in the next dev iteration depending on effort. Either |
The "Infinite" and "Invalid Date" problems will be solved already in 10.0.8, I touched that bit when fixing "retry chunks" in #30776 |
Decision for this depends very much on the use case and requirements. |
https://github.com/websanova/vue-upload -> 11.0 -> Phoenix hope this is sufficient for most users from a timeframe perspective. |
if this is a full fledge uploader we'll likely need to hack it to make chunking work if this is just an indicator that we can plug into our own upload code then this is better |
I guess what I wanted to say: There is proper code for frontends around which we can reuse in the future. Not a prio for today. |
Steps to reproduce:
Expected behaviour:
Time left written in the progress bar decreases per second.
Actual behaviour
Time left is variable, it can increase a lot, decrease, increase again.
I recommend to remove this text from progress bar or improve the estimation.
cc: @luckydonald
The text was updated successfully, but these errors were encountered: