fix(api): improve the error msg returned to UI (#6275)

sguiheux · web-flow · commit 8913af3d96f5 · 2022-09-09T14:54:41.000+02:00
diff --git a/sdk/exportentities/v2/workflow.go b/sdk/exportentities/v2/workflow.go
@@ -604,7 +604,7 @@ func (e *NodeEntry) processNodeAncestors(name string, w *sdk.Workflow) (bool, er
 	case 0:
 		// If there is already a root node, it is impossible have another one
 		if w.WorkflowData.Node.Name != "" {
-			return false, fmt.Errorf("invalid node dependencies. %s should have at least one dependency because the workflow already have a root", n.Name)
+			return false, sdk.NewErrorFrom(sdk.ErrWrongRequest, "invalid node dependencies. %s should have at least one dependency because the workflow already have a root", n.Name)
 		}
 		w.WorkflowData.Node = *n
 		return true, nil

Original file line number	Diff line number	Diff line change
`@@ -604,7 +604,7 @@ func (e NodeEntry) processNodeAncestors(name string, w sdk.Workflow) (bool, er`
`604`	`604`	`case 0:`
`605`	`605`	`// If there is already a root node, it is impossible have another one`
`606`	`606`	`if w.WorkflowData.Node.Name != "" {`
`607`		`- return false, fmt.Errorf("invalid node dependencies. %s should have at least one dependency because the workflow already have a root", n.Name)`
	`607`	`+ return false, sdk.NewErrorFrom(sdk.ErrWrongRequest, "invalid node dependencies. %s should have at least one dependency because the workflow already have a root", n.Name)`
`608`	`608`	`}`
`609`	`609`	`w.WorkflowData.Node = *n`
`610`	`610`	`return true, nil`