search

mskcc / LimsRest

the restful service used by the IGO LIMS
2 stars 2 forks source link

Fixing case of descendant SeqAnalysisSampleQCModel DataRecords #192

Closed DavidStreid closed 4 years ago

DavidStreid commented 4 years ago

Change: Check all SeqAnalysisSampleQCModel descendants from input Sample DataRecord instead of just the first SeqAnalysisSampleQCModel child

A Sample DataRecord can have a SeqAnalysisSampleQCModel child AND have children Sample DataRecords that also have SeqAnalysisSampleQCModel children. Therefore, we need to grab all descendant SeqAnalysisSampleQCModel from the input Sample DataRecord, @record, and check to see if any of them have been marked as IGO-Complete. This is because as long as one child has successfully completed the sequencing workflow, this WorkflowSample should be marked as Complete and NOT failed

mskbot commented 4 years ago

Can one of the admins verify this patch?