Skip to content

Commit 73e49c1

Browse files
Merge pull request #609 from onflow/bugfix/flip-enable-metrics
Fix bug for setting metric settings
2 parents 119d318 + 04b0dfa commit 73e49c1

File tree

4 files changed

+9
-8
lines changed

4 files changed

+9
-8
lines changed

.github/workflows/cd.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@ jobs:
1616
go-version: '1.18'
1717
env:
1818
SNYK_TOKEN: ${{ secrets.SNYK_TOKEN }}
19-
MIXPANEL_PROJECT_TOKEN: ${{ secrets.MIXPANEL_PROJECT_TOKEN }}
20-
MIXPANEL_SERVICE_ACCOUNT_SECRET: ${{ secrets.MIXPANEL_SERVICE_ACCOUNT_SECRET }}
2119
build:
2220
runs-on: ubuntu-latest
2321
steps:
@@ -29,6 +27,9 @@ jobs:
2927
go-version: '1.18'
3028
- name: Build Binaries
3129
run: make versioned-binaries
30+
env:
31+
MIXPANEL_PROJECT_TOKEN: ${{ secrets.MIXPANEL_PROJECT_TOKEN }}
32+
MIXPANEL_SERVICE_ACCOUNT_SECRET: ${{ secrets.MIXPANEL_SERVICE_ACCOUNT_SECRET }}
3233
- name: Uploading Binaries
3334
uses: google-github-actions/upload-cloud-storage@main
3435
with:

Makefile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,14 +56,14 @@ install:
5656
GO111MODULE=on go install \
5757
-trimpath \
5858
-ldflags \
59-
"-X github.com/onflow/flow-cli/build.commit=$(COMMIT) -X github.com/onflow/flow-cli/build.semver=$(VERSION) -X github.com/onflow/flow-cli/pkg/flowkit/util.MIXPANEL_SERVICE_ACCOUNT_SECRET=$(MIXPANEL_SERVICE_ACCOUNT_SECRET) -X github.com/onflow/flow-cli/pkg/flowkit/util.MIXPANEL_PROJECT_TOKEN=$(MIXPANEL_PROJECT_TOKEN)" \
59+
"-X github.com/onflow/flow-cli/build.commit=$(COMMIT) -X github.com/onflow/flow-cli/build.semver=$(VERSION) -X github.com/onflow/flow-cli/pkg/flowkit/util.MIXPANEL_SERVICE_ACCOUNT_SECRET=${MIXPANEL_SERVICE_ACCOUNT_SECRET} -X github.com/onflow/flow-cli/pkg/flowkit/util.MIXPANEL_PROJECT_TOKEN=${MIXPANEL_PROJECT_TOKEN}" \
6060
./cmd/flow
6161

6262
$(BINARY):
6363
GO111MODULE=on go build \
6464
-trimpath \
6565
-ldflags \
66-
"-X github.com/onflow/flow-cli/build.commit=$(COMMIT) -X github.com/onflow/flow-cli/build.semver=$(VERSION) -X github.com/onflow/flow-cli/pkg/flowkit/util.MIXPANEL_SERVICE_ACCOUNT_SECRET=$(MIXPANEL_SERVICE_ACCOUNT_SECRET) -X github.com/onflow/flow-cli/pkg/flowkit/util.MIXPANEL_PROJECT_TOKEN=$(MIXPANEL_PROJECT_TOKEN)"\
66+
"-X github.com/onflow/flow-cli/build.commit=$(COMMIT) -X github.com/onflow/flow-cli/build.semver=$(VERSION) -X github.com/onflow/flow-cli/pkg/flowkit/util.MIXPANEL_SERVICE_ACCOUNT_SECRET=${MIXPANEL_SERVICE_ACCOUNT_SECRET} -X github.com/onflow/flow-cli/pkg/flowkit/util.MIXPANEL_PROJECT_TOKEN=${MIXPANEL_PROJECT_TOKEN}"\
6767
-o $(BINARY) ./cmd/flow
6868

6969
.PHONY: versioned-binaries
@@ -105,4 +105,4 @@ check-headers:
105105
.PHONY: check-tidy
106106
check-tidy:
107107
go mod tidy
108-
cd pkg/flowkit; go mod tidy
108+
cd pkg/flowkit; go mod tidy

internal/config/metrics-settings.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,8 @@ func handleMetricsSettings(
5151
_ command.GlobalFlags,
5252
_ *services.Services,
5353
) (command.Result, error) {
54-
disabled := args[0] == "disable"
55-
err := util.SetUserMetricsSettings(disabled)
54+
enabled := args[0] == "enable"
55+
err := util.SetUserMetricsSettings(enabled)
5656
if err != nil {
5757
return nil, err
5858
}

pkg/flowkit/util/mixpanel.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ import (
3131

3232
const (
3333
MIXPANEL_TRACK_URL = "https://api.mixpanel.com/track"
34-
MIXPANEL_QUERY_URL = "https://mixpanel.com/api/2.0/engage?project_id=2154593"
34+
MIXPANEL_QUERY_URL = "https://mixpanel.com/api/2.0/engage?project_id=2123215"
3535
MIXPANEL_PROFILE_URL = "https://api.mixpanel.com/engage#profile-set"
3636
)
3737

0 commit comments

Comments
 (0)