Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Zip File Download Error Messaging: You shouldn't have to unzip the archive and read a manifest file to know there was an error #2060

Closed
posixeleni opened this issue Apr 21, 2015 · 9 comments
Closed

Zip File Download Error Messaging: You shouldn't have to unzip the archive and read a manifest file to know there was an error #2060

posixeleni opened this issue Apr 21, 2015 · 9 comments
Labels
Feature: File Upload & Handling Type: Feature a feature request User Role: Guest Anyone using the system, even without an account UX & UI: Design This issue needs input on the design of the UI and from the product owner

Comments

@posixeleni
Copy link
Contributor

Reported by a user https://twitter.com/AABoyles/status/590509840532971520 that the file download error messaging only appears if you open up the manifest file. There should be clearer error messaging directly in the UI if a single file or a group of selected files fail to download.
@posixeleni posixeleni added UX & UI: Design This issue needs input on the design of the UI and from the product owner Type: Suggestion an idea Feature: File Upload & Handling labels Apr 21, 2015
@scolapasta scolapasta added this to the 4.0.1 milestone Apr 21, 2015
@eaquigley
Copy link
Contributor

@posixeleni is this happening only with files that have been zipped or is this for all files that aren't properly downloading?

@posixeleni
Copy link
Contributor Author

@eaquigley from this users error this looks to be only about error messaging for zipped file downloads but I am not sure about error messaging for individual file downloads.

@eaquigley
Copy link
Contributor

@landreev can you answer if the file download error message is only in the manifest for a zip file that fails? or if all error messages for file download behave this way.

@scolapasta scolapasta modified the milestones: Candidates for 4.0.1, Design May 8, 2015
@eaquigley eaquigley modified the milestones: In Design, In Review May 8, 2015
@eaquigley eaquigley modified the milestones: 4.2, In Review Aug 10, 2015
@eaquigley
Copy link
Contributor

Moving into 4.2 as downloading multiple files is scheduled to occur during this milestone. Error message currently displaying only in the readme file for the downloads, needs to also display in the UI when a user attempts to download multiple files and it is too big to allow them to unselect files and try downloading without having to wait to open the readme file.

@eaquigley eaquigley modified the milestones: 4.2, In Design Aug 14, 2015
@eaquigley
Copy link
Contributor

Mockups for this found here: https://iqssharvard.mybalsamiq.com/projects/filesallthings-enduser-dataverse42andbeyond/Dataset%20-%20Files%20-%20Download%20Limit%20Error

Passing to @sekmiller

@eaquigley eaquigley modified the milestones: 4.3, In Design Sep 25, 2015
@eaquigley eaquigley assigned sekmiller and unassigned eaquigley Sep 25, 2015
@eaquigley eaquigley removed the Type: Suggestion an idea label Sep 25, 2015
@mercecrosas mercecrosas modified the milestones: 4.3, In Review Nov 30, 2015
@sekmiller sekmiller removed their assignment Jan 28, 2016
@scolapasta scolapasta removed this from the Not Assigned to a Release milestone Jan 28, 2016
@mheppler
Copy link
Contributor

mheppler commented Feb 3, 2016

I was able to replicate this locally. The issue relates to the "Download" button at the top of the files table on the dataset page not properly handling when restricted files are selected, whether it's multiple files or a single file. The same problem has been reported by @sbarbosadataverse in #2722.

@eaquigley eaquigley assigned djbrooke and unassigned eaquigley Aug 1, 2016
@djbrooke djbrooke removed their assignment Aug 2, 2016
@pdurbin pdurbin mentioned this issue Sep 15, 2016
Closed
@pdurbin
Copy link
Member

pdurbin commented Dec 12, 2016

This bug was reported at https://twitter.com/justin_littman/status/803929260759154688 a couple weeks ago.

@pdurbin pdurbin changed the title File Download Error Messaging Zip File Download Error Messaging: You shouldn't have to unzip the archive and read a manifest file to know there was an error Dec 21, 2016
@kcondon
Copy link
Contributor

kcondon commented May 24, 2017

Reported again today, see RT 249731

@pdurbin pdurbin added User Role: Guest Anyone using the system, even without an account and removed zTriaged labels Jun 30, 2017
@pdurbin pdurbin mentioned this issue Mar 14, 2018
Closed
@pdurbin
Copy link
Member

pdurbin commented Apr 9, 2018 •
edited
Loading

I'm closing this in favor of #4576.

@pdurbin pdurbin closed this as completed Apr 9, 2018
@pdurbin pdurbin mentioned this issue Apr 10, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature: File Upload & Handling Type: Feature a feature request User Role: Guest Anyone using the system, even without an account UX & UI: Design This issue needs input on the design of the UI and from the product owner
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
9 participants
@pdurbin @mercecrosas @mheppler @sekmiller @scolapasta @kcondon @posixeleni @eaquigley @djbrooke