fix(workflows): make projectId required when listing workflows #372

Roger Lam · web-flow · commit d31ed8a0302b · 2021-09-29T16:53:56.000-04:00
Results weren't returning if projectId wasn't provided. This makes the field required so there isn't confusion if not results are returned.
diff --git a/gradient/cli/workflows.py b/gradient/cli/workflows.py
@@ -79,7 +79,7 @@ def create_workflow_run(ctx, api_key, workflow_id, cluster_id, spec_path, input_
 @click.option(
     "--projectId",
     "project_id",
-    required=False,
+    required=True,
     help="Project ID",
     cls=common.GradientOption,
 )

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ def create_workflow_run(ctx, api_key, workflow_id, cluster_id, spec_path, input_`
`79`	`79`	`@click.option(`
`80`	`80`	`"--projectId",`
`81`	`81`	`"project_id",`
`82`		`- required=False,`
	`82`	`+ required=True,`
`83`	`83`	`help="Project ID",`
`84`	`84`	`cls=common.GradientOption,`
`85`	`85`	`)`