-
Couldn't load subscription status.
- Fork 79
WIP: Email status upgrade #3242
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: Email status upgrade #3242
Conversation
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.
|
@antgonza includes changes from the other PR. I believe they will disappear when the other PR is merged into dev branch? |
|
That's correct but this and the other issue (AFAIK) do not touch the same files so, IMOO, it's better to keep them separate by starting both with a clean dev env; hopefully next time. |
|
@charles-cowart, note that as there were changes in the Anyway, can you fix conflicts here or open a new clean PR? Thank you. |
|
|
||
| # generate message line | ||
| input_artifacts = self.input_artifacts() | ||
| if input_artifacts is None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consider revising. not sure if None is a good condition to test against.
|
Closing. Superseded by #3230 |
No description provided.