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.

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

LIMS-1469: Fix download button on old summary page #840

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions api/src/Page/Download.php
Original file line number Diff line number Diff line change
Expand Up @@ -91,9 +91,7 @@ function _download_visit()
$this->_error('There doesnt seem to be a data archive available for this visit');
}

# ------------------------------------------------------------------------
# Download mtz/log file for Fast DP / XIA2
# TODO: Delete me

# This method either returns a list of plots from MX auto processing tools (n_obs, n_uniq, completeness etc.)
# Or returns a specific plot based on auto processing attachment id (aid).
# Individual plotly format Graphs can be returned via an aid, but will not be included in the list of plots (as their format is different)
Expand Down
2 changes: 1 addition & 1 deletion client/src/js/modules/dc/views/summary.js
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ define(['marionette',
{ label: 'Resolution', cell: APCell, template: '<%-SHELLS.overall.RLOW%> - <%-SHELLS.overall.RHIGH%><br /><%-SHELLS.innerShell.RLOW%> - <%-SHELLS.innerShell.RHIGH%><br /><%-SHELLS.outerShell.RLOW%> - <%-SHELLS.outerShell.RHIGH%>', editable: false },
{ label: 'Rmeas', cell: APCell, template: '<span class="<%-CLASS.RMEAS.overall%>"><%-SHELLS.overall.RMEAS%></span><br /><span class="<%-CLASS.RMEAS.innerShell%>"><%-SHELLS.innerShell.RMEAS%></span><br /><span class="<%-CLASS.RMEAS.outerShell%>"><%-SHELLS.outerShell.RMEAS%></span>', editable: false },
{ label: 'Completeness', cell: APCell, template: '<span class="<%-CLASS.COMPLETENESS.overall%>"><%-SHELLS.overall.COMPLETENESS%></span><br /><span class="<%-CLASS.COMPLETENESS.innerShell%>"><%-SHELLS.innerShell.COMPLETENESS%></span><br /><span class="<%-CLASS.COMPLETENESS.outerShell%>"><%-SHELLS.outerShell.COMPLETENESS%></span>', editable: false },
{ label: '', cell: APCell, template: '<a href="'+app.apiurl+'/download/id/<%-DCID%>/aid/<%-AID%>" class="button button-notext dll" title="Download MTZ file"><i class="fa fa-download"></i> <span>Download MTZ file</span></a>', editable: false },
{ label: '', cell: APCell, template: '<a href="'+app.apiurl+'/download/ap/archive/<%-AID%>" class="button button-notext dll" title="Download autoprocessing archive"><i class="fa fa-download"></i> <span>Download autoprocessing archive</span></a>', editable: false },

]

Expand Down
4 changes: 2 additions & 2 deletions client/src/js/modules/summary/views/summary.vue
Original file line number Diff line number Diff line change
Expand Up @@ -397,8 +397,8 @@
<i class="fa fa-search"></i></a>
</p>
<p class="tw-ra tw-ml-1">
<a href="'+app.apiurl+'/download/id/<%-DCID%>/aid/<%-AID%>" class="tw-button tw-button-notext tw-dll"
title="Download MTZ file"><i class="fa fa-download"></i></a>
<a href="'+app.apiurl+'/download/ap/archive/<%-AID%>" class="tw-button tw-button-notext tw-dll"
title="Download autoprocessing archive"><i class="fa fa-download"></i></a>
</p> -->


Expand Down
Loading