Merge pull request #723 from Project-MONAI/AW-RevertClinicalReviewTaskModifications

Reverted changes for Workflow validation

Author: woodheadio

src/Shared/Shared/ValidationConstants.cs

@@ -48,24 +48,13 @@ public static class ValidationConstants
         /// </summary>
         public static readonly string Mode = "mode";
 
-        /// <summary>
-        /// Key for the notifications.
-        /// </summary>
-        public static readonly string Notifications = "notifications";
-
         public enum ModeValues
         {
             QA,
             Research,
             Clinical
         }
 
-        public enum NotificationValues
-        {
-            True,
-            False
-        }
-
         /// <summary>
         /// Required arguments to run the clinical review task workflow args.
         /// </summary>

src/WorkflowManager/WorkflowManager/Validators/WorkflowValidator.cs

@@ -407,16 +407,6 @@ private void ValidateClinicalReviewRequiredFields(TaskObject[] tasks, TaskObject
                 return;
             }
 
-            if (!currentTask.Args.ContainsKey(Notifications))
-            {
-                Errors.Add($"Task: '{currentTask.Id}' notifications must be specified.");
-                return;
-            }
-            else if (!Enum.TryParse(typeof(NotificationValues), currentTask.Args[Notifications], true, out var _))
-            {
-                Errors.Add($"Task: '{currentTask.Id}' notifications is incorrectly specified{Comma}please specify 'true' or 'false'");
-            }
-
             var reviewedTask = tasks.First(t => t.Id.ToLower() == currentTask.Args[ReviewedTaskId].ToLower());
 
             if (reviewedTask.Type.Equals(ArgoTaskType, StringComparison.OrdinalIgnoreCase) is false)