Merge pull request #1129 from CedricFinance/master

Fix WorkflowStepCompletedRequest options handling
slack-go · Dec 2, 2022 · 93341d4 · 93341d4
2 parents c99b47b + 3d6bc00
commit 93341d4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/workflow_step_execute.go b/workflow_step_execute.go
@@ -19,10 +19,10 @@ type (
 	}
 )
 
-type WorkflowStepCompletedRequestOption func(opt WorkflowStepCompletedRequest) error
+type WorkflowStepCompletedRequestOption func(opt *WorkflowStepCompletedRequest) error
 
 func WorkflowStepCompletedRequestOptionOutput(outputs map[string]string) WorkflowStepCompletedRequestOption {
-	return func(opt WorkflowStepCompletedRequest) error {
+	return func(opt *WorkflowStepCompletedRequest) error {
 		if len(outputs) > 0 {
 			opt.Outputs = outputs
 		}
@@ -33,7 +33,7 @@ func WorkflowStepCompletedRequestOptionOutput(outputs map[string]string) Workflo
 // WorkflowStepCompleted indicates step is completed
 func (api *Client) WorkflowStepCompleted(workflowStepExecuteID string, options ...WorkflowStepCompletedRequestOption) error {
 	// More information: https://api.slack.com/methods/workflows.stepCompleted
-	r := WorkflowStepCompletedRequest{
+	r := &WorkflowStepCompletedRequest{
 		WorkflowStepExecuteID: workflowStepExecuteID,
 	}
 	for _, option := range options {