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

WIP: Fix 1805 #2062

Closed
wants to merge 8 commits into from
Closed

WIP: Fix 1805 #2062

wants to merge 8 commits into from

Conversation

antgonza
Copy link
Member

@antgonza antgonza commented Jan 29, 2017

Based on #2059, so merge that one first.
The idea is, as defined in the issue, to download all file paths from the BIOM artifacts that the user has access to via nginx zip.

ToDo:

  • figure out how to test failures, place holders in place
  • test/tweak in the test-env until it actually works as expected

The new button is show here:
screen shot 2017-01-28 at 6 32 49 pm

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 91.611% when pulling 1626859 on antgonza:fix-1805 into 4e380e0 on biocore:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 91.625% when pulling ade8cbe on antgonza:fix-1805 into 4e380e0 on biocore:master.

@antgonza antgonza changed the title Fix 1805 WIP: Fix 1805 Feb 3, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 91.64% when pulling 890d465 on antgonza:fix-1805 into 2795046 on biocore:master.

@josenavas
Copy link
Contributor

Closing in favor of #2087

@josenavas josenavas closed this Mar 22, 2017
@antgonza antgonza mentioned this pull request Mar 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants