Now, in the detail of the SamplesReport, a new button is displayed to download the NIH files:
As the issue specifies, when clicking the button the following modal is shown:
This triggers the download of a zip file, which at the moment has only the file Instructions.txt (the other files will be generated in further issues). This file is empty and its contents will be defined together with the client (it needs to be uploaded to a specific portal and we need to collect this info).
NOTE:
In the ConfirmationModal, the X button to close it, calls the hide() method of the <Modal> component that it renders, but this is not how the ConfirmationModal is used: it has a hidden class that is added or removed as needed. To avoid problems with the other uses of <Modal> in the project, a new option for the confirmation modal was added, showCloseButton, settled as false in all the uses. This fixes a bug, since when ConfirmationModal was used and closed with the X and not with the Cancel button, it was breaking its functioning.
Closes #1917.
Now, in the detail of the
SamplesReport
, a new button is displayed to download the NIH files:As the issue specifies, when clicking the button the following modal is shown:
This triggers the download of a zip file, which at the moment has only the file
Instructions.txt
(the other files will be generated in further issues). This file is empty and its contents will be defined together with the client (it needs to be uploaded to a specific portal and we need to collect this info).NOTE: In the
ConfirmationModal
, theX
button to close it, calls thehide()
method of the<Modal>
component that it renders, but this is not how theConfirmationModal
is used: it has ahidden
class that is added or removed as needed. To avoid problems with the other uses of<Modal>
in the project, a new option for the confirmation modal was added,showCloseButton
, settled as false in all the uses. This fixes a bug, since whenConfirmationModal
was used and closed with theX
and not with theCancel
button, it was breaking its functioning.