Mccalluc/details tab

[mccalluc]

Towards #1890.

Two questions about the old one, copied from the thread there:

Missing None:None under STUDIES

This does not come through because in the source code we have

  <span ng-if="study.identifier">
    <strong>{{study.identifier}}</strong>
  </span>
  <span ng-if="study.title">
    : {{study.title}}
  </span>

... which seems more like a feature than a bug?

Also: New one is missing creation date of the current version: Should that also be ported from the old version?

Import into Own Space might be more complicated, but this much could be merged, I think.

original:

with this pr:

[scottx611x]

Looks good to me, check out

refinery-platform/refinery/templates/core/data_set.html

Lines 585 to 595 in 0701a35

	{% if isatab_archive and data_set.is_public and data_set.get_owner_username != user.username and user.id %}
	<form id="isaTabImportForm" enctype="multipart/form-data" action="{% url "take_ownership_of_public_dataset" %}" method="post">
	{% csrf_token %}
	<p>
	<input id="id_isa_tab_url" type="hidden" name="isa_tab_url" value="{{ isatab_archive.get_datafile_url }}" />
	</p>
	<input class="refinery-base btn btn-important"
	id="isa-import-button" type="submit" value="Import into own Space"/>
	<div style="padding: 15px; display: block; height: 22px; width: 22px;" id="isa-spinner"></div>
	</form>
	{% endif %}

for Import into own space code

[mccalluc]

@scottx611x : Thanks: If you have thoughts on the two questions at the top, let me know?

[codecov-io]

Codecov Report

Merging #2060 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2060   +/-   ##
========================================
  Coverage    44.22%   44.22%           
========================================
  Files          417      417           
  Lines        27729    27729           
  Branches      1340     1340           
========================================
  Hits         12263    12263           
  Misses       15466    15466

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b28b996...854eb4f. Read the comment docs.

[scottx611x]

@mccalluc regarding the None:None i'm not sure if we want to remove the ng-ifs to keep consistent with the functionality that we had before? It seems odd to me to explicitly state the Nones, though

As for the creation date of the newest version I think that would be great to include.