Skip to content

Commit aecf41f

Browse files
committed
Merge pull request #11 from optimizely/generate_goal_names
Generates Goal Names correctly
2 parents 4fef71d + 71f2cf5 commit aecf41f

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

optimizely/optimizely.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,12 +93,13 @@ func (client *OptimizelyClient) Activate(experiment_key string, user_id string,
9393
}
9494

9595
end_user_id := fmt.Sprintf(END_USER_ID_TEMPLATE, user_id)
96-
//variation_id := client.Bucket(experiment_key, user_id)
96+
variation_id := client.Bucket(experiment_key, user_id)
9797

9898
parameters := url.Values{}
9999
parameters.Add(ACCOUNT_ID, client.account_id)
100100
parameters.Add(PROJECT_ID, client.project_config.ProjectId)
101-
parameters.Add(GOAL_NAME, "TODO")
101+
parameters.Add(GOAL_NAME, GenerateGoalName(client.project_config.Events))
102+
parameters.Add(fmt.Sprintf("{%v}{%v}", EXPERIMENT, experiment_id), variation_id)
102103
parameters.Add(GOAL_ID, experiment_id)
103104
parameters.Add(END_USER_ID, end_user_id)
104105

0 commit comments

Comments
 (0)