Skip to content

Fix #3222 #3241

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

Merged
merged 10 commits into from
Jan 30, 2023
Merged

Fix #3222 #3241

merged 10 commits into from
Jan 30, 2023

Conversation

charles-cowart
Copy link
Contributor

No description provided.

This will make it easier to track back to individual artifacts.
When an artifact is deleted, files associated with the artifact are
moved from a location associated with the artifact back into the study
uploads folder. The following two files:
    'qtp-sequencing-validate-data.csv'
    'feature-table.qza'
should be deleted instead.
@charles-cowart charles-cowart changed the title Del artifact fix v2 Fix #3222 Jan 27, 2023
@coveralls
Copy link

coveralls commented Jan 27, 2023

Coverage Status

coverage: 92.921% (+0.005%) from 92.916%
when pulling ef6e36a on charles-cowart:del_artifact_fix_v2
into 16f415c on qiita-spots:dev.

@antgonza
Copy link
Member

Thank you.

@antgonza antgonza merged commit 6684ad0 into qiita-spots:dev Jan 30, 2023
@charles-cowart charles-cowart deleted the del_artifact_fix_v2 branch March 1, 2023 01:22
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.

Deleting artifacts should not move logs/summaries to the upload folder
3 participants