-
Notifications
You must be signed in to change notification settings - Fork 6
Download should be incremented as soon as you click on downloads #1120
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
Download should be incremented as soon as you click on downloads #1120
Conversation
|
I am testing this for datasets and files. With datasets, when I click the download button the download increments, but the dataset did not download. This happens both on the regular dataset view and the public dataset view. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think download works. I see it in the network with the binaries; but since you are not catch the binary and save it to blob then generate a tag, the binary is not accessible to end user.
The public download doesn't work for the same reason.
We used to take similar approach as I described before, but at one point we switch to just embed the url and let the browser handle download due to large files. Not sure if we would like to switch back. Maybe something we can discuss?
…ads" This reverts commit b8d7e11.
…ntside on clicking download button
|
this is fixed. @tcnichol @longshuicy feel free to take a look |
|
Tested. From the dataset page, every click of download increments. Same for files. If I download datasets from 'explore' download also increments. Same with downloading files from the filelist. Marking this one approved. |
tcnichol
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Download increments on datasets and files from any location. Marking this approved.
longshuicy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works well. Thanks!
No description provided.