Merge pull request #614 from onflow/bugfix/enable-tracking-default

sukantoraymond · web-flow · commit aa3fd2690e7d · 2022-08-12T12:51:28.000-04:00
Enable tracking by default if there is rate limit error from mixpanel
diff --git a/pkg/flowkit/util/mixpanel.go b/pkg/flowkit/util/mixpanel.go
@@ -160,6 +160,10 @@ func IsUserOptedIn() (bool, error) {
 		return false, err
 	}
 	if res.StatusCode >= 400 {
+		if res.StatusCode == 429 {
+			//tracking is enabled by default if there is rate limit error from mixpanel
+			return true, nil
+		}
 		return false, fmt.Errorf("invalid response status code %d for tracking command usage", res.StatusCode)
 	}
 	if len(queryResponse.Results) == 0 {

Original file line number	Diff line number	Diff line change
`@@ -160,6 +160,10 @@ func IsUserOptedIn() (bool, error) {`
`160`	`160`	`return false, err`
`161`	`161`	`}`
`162`	`162`	`if res.StatusCode >= 400 {`
	`163`	`+ if res.StatusCode == 429 {`
	`164`	`+ //tracking is enabled by default if there is rate limit error from mixpanel`
	`165`	`+ return true, nil`
	`166`	`+ }`
`163`	`167`	`return false, fmt.Errorf("invalid response status code %d for tracking command usage", res.StatusCode)`
`164`	`168`	`}`
`165`	`169`	`if len(queryResponse.Results) == 0 {`