stable
Clone or download
Part of story #12404 download folder as zip Since the modal is always opened, it is likely that people will click it without reading when a problem does arise and there is actually a TULEAP_ERRORS.txt file. We will instead send an email (in another commit). How to test: - make tests_cypress should pass - when you download a zip folder with size inferior to the "warning limit", there won't be any modal. - Modals are still shown for "warning limit", "OSX warning" and "Max size limit" cases. Note: This solution brings back the Chromium warning: Resource interpreted as Document but transferred with MIME type application/x-zip But it can't be worked around. If we put the download URI as href in the link, people will be able to bypass the "size exceeded" modal with ctrl+click. Change-Id: I04758efcc8a6a218a8435234112ef18b10791b96
Modified Files
Name | ||||
---|---|---|---|---|
M | plugins/document/scripts/document/components/Folder/DropDown/DownloadFolderAsZip/DownloadFolderAsZip.test.js | +13 | −17 | Go to diff View file |
M | plugins/document/scripts/document/components/Folder/DropDown/DownloadFolderAsZip/DownloadFolderAsZip.vue | +2 | −5 | Go to diff View file |
M | plugins/document/scripts/document/components/Folder/DropDown/DownloadFolderAsZip/ModalArchiveSizeWarning.vue | +0 | −11 | Go to diff View file |
D | plugins/document/scripts/document/components/Folder/DropDown/DownloadFolderAsZip/ModalConfirmArchiveDownload.test.js | +0 | −63 | Go to diff View file |
D | plugins/document/scripts/document/components/Folder/DropDown/DownloadFolderAsZip/ModalConfirmArchiveDownload.vue | +0 | −100 | Go to diff View file |
D | plugins/document/scripts/document/components/Folder/DropDown/DownloadFolderAsZip/WarningAboutArchiveErrors.vue | +0 | −10 | Go to diff View file |
M | plugins/document/scripts/document/components/Folder/FolderHeader.test.js | +0 | −20 | Go to diff View file |
M | plugins/document/scripts/document/components/Folder/FolderHeader.vue | +0 | −18 | Go to diff View file |
M | plugins/document/scripts/document/po/fr.po | +0 | −29 | Go to diff View file |
M | plugins/document/tests/e2e/cypress/cypress/integration/document.spec.js | +21 | −25 | Go to diff View file |