Workflows: Fix error source when unexpected time field is used (#437)

ivanahuckova · web-flow · commit 1bca5a53f803 · 2025-03-14T15:01:32.000+01:00
We support 2 time fields and use `0` and `1` to distinguish them. If user uses any other time fields `2`, `3`, ... this causes an error as it is not supported. The error should be downstream. <img width="636" alt="image" src="https://github.com/user-attachments/assets/96bd3b67-1260-421d-bdb4-3276d7d620cb" /> After fix: <img width="1506" alt="image" src="https://github.com/user-attachments/assets/7ca6f2f1-beb2-43dd-9674-e8ba45eb50ec" />
diff --git a/pkg/github/workflows.go b/pkg/github/workflows.go
@@ -88,7 +88,7 @@ func keepWorkflowsInTimeRange(workflows []*googlegithub.Workflow, timeField mode
 			}
 
 		default:
-			return nil, fmt.Errorf("unexpected time field: %d", timeField)
+			return nil, backend.DownstreamError(fmt.Errorf("unexpected time field: %d", timeField))
 		}
 
 		out = append(out, workflow)

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ func keepWorkflowsInTimeRange(workflows []*googlegithub.Workflow, timeField mode`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`default:`
`91`		`- return nil, fmt.Errorf("unexpected time field: %d", timeField)`
	`91`	`+ return nil, backend.DownstreamError(fmt.Errorf("unexpected time field: %d", timeField))`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`out = append(out, workflow)`