Project

General

Profile

Actions

Bug #17289

closed

Download of folders in collections should be disabled

Added by Daniel Kutyła almost 4 years ago. Updated 7 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Target version:
Story points:
-
Release relationship:
Auto

Description

When selecting "Download" from the menu next to a folder in a collection the result is that a html file is downloaded. This is the keep-web html that otherwise is displayed when selecting "Open in new tab".

Expected behaviour: A zip is created containing the folder with all nested files and subfolders. This should also reliably work on subfolders.

Workaround: The expected behaviour requires an API doing this on the server which also works for large datasets. Until this is implemented the download mechanism should be disabled for everything that is not a file.

Side note: The instructions to download collection content (keep-web HTML) contain a wget command which does not contain authentication. Without this is non-functional.


Subtasks 1 (0 open1 closed)

Task #17596: Review 17289-dir-download-action-removalResolvedPeter Amstutz04/28/2021Actions
Actions #1

Updated by Peter Amstutz over 3 years ago

  • Target version set to 2021-05-12 sprint
Actions #2

Updated by Peter Amstutz over 3 years ago

  • Assigned To set to Lucas Di Pentima
Actions #3

Updated by Lucas Di Pentima over 3 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Lucas Di Pentima over 3 years ago

Updates at arvados-workbench2|beacdc4d - branch 17289-dir-download-action-removal
Test run: developer-tests-workbench2: #398

  • Removes "Download" action from directories on collection files listing.
  • Adds test.
Actions #5

Updated by Peter Amstutz over 3 years ago

Reviewing 17289-dir-download-action-removal @ beacdc4d38efb1880aaa14e8b3985e683d14e457

In src/views-components/context-menu/action-sets/collection-files-item-action-set.ts line 53-61 is commented out code that probably doesn't need to be there? Or a comment about why it is commented out and why you might want it later.

Rest LGTM.

Actions #6

Updated by Lucas Di Pentima over 3 years ago

Thanks, that commented-out code was other attempts that I was trying, removed and merged!

Actions #7

Updated by Anonymous over 3 years ago

  • Status changed from In Progress to Resolved
Actions #8

Updated by Peter Amstutz 7 months ago

  • Release set to 38
Actions

Also available in: Atom PDF