Skip to content

Commit

Permalink
Resolve Node IDs in dependent steps, even on partial successes
Browse files Browse the repository at this point in the history
  • Loading branch information
JohnStarich committed Aug 21, 2024
1 parent 4532b50 commit e547bb4
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions execute.go
Original file line number Diff line number Diff line change
Expand Up @@ -256,16 +256,12 @@ func executeOneStep(
}

// fire the query
err := queryer.Query(ctx.RequestContext, &graphql.QueryInput{
queryErr := queryer.Query(ctx.RequestContext, &graphql.QueryInput{
Query: step.QueryString,
QueryDocument: step.QueryDocument,
Variables: variables,
OperationName: operationName,
}, &queryResult)
if err != nil {
ctx.logger.Warn("Network Error: ", err)
return queryResult, nil, err
}

// NOTE: this insertion point could point to a list of values. If it did, we have to have
// passed it to the this invocation of this function. It is safe to trust this
Expand Down Expand Up @@ -313,7 +309,7 @@ func executeOneStep(
}
}
}
return queryResult, dependentSteps, nil
return queryResult, dependentSteps, queryErr
}

func max(a, b int) int {
Expand Down

0 comments on commit e547bb4

Please sign in to comment.