It would seem our documentation is inaccurate BA-5828 (broadinstitute…

…#5074)
AlekseiLitkovetc · Jul 18, 2019 · 295005f · 295005f
1 parent 6ab134d
commit 295005f
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/services/src/main/scala/cromwell/services/metadata/WorkflowQueryKey.scala b/services/src/main/scala/cromwell/services/metadata/WorkflowQueryKey.scala
@@ -134,6 +134,10 @@ object WorkflowQueryKey {
 
     override def validate(grouped: Map[String, Seq[(String, String)]]): ErrorOr[List[String]] = {
       val values = valuesFromMap(grouped).toList
+      /*
+        The inclusion of `WorkflowMetadataKeys.ParentWorkflowId` is for backwards compatibility. As of #4381
+        parentWorkflowId is always included, but we did not want to break old automated queries
+        */
       val allowedValues = Seq(WorkflowMetadataKeys.Labels, WorkflowMetadataKeys.ParentWorkflowId)
       val nels: List[ErrorOr[String]] = values map { v => {
         allowedValues.contains(v).fold(v.validNel[String], v.invalidNel[String])