fix(gacli): fix analytics event name (#3860)

* fix analytics event name * use cmd with _ * add changelog --------- Co-authored-by: Pantani <Pantani>
ignite · Dec 21, 2023 · 7d39994 · 7d39994
1 parent 3dda9b0
commit 7d39994
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/changelog.md b/changelog.md
@@ -23,6 +23,7 @@
 - [#3833](https://github.com/ignite/cli/pull/3833) Improve Cosmos SDK detection to support SDK forks
 - [#3849](https://github.com/ignite/cli/pull/3849) Add missing `tx.go` file by default and enable cli if autocli does not exist
 - [#3851](https://github.com/ignite/cli/pull/3851) Add missing ibc interfaces to chain client
+- [#3860](https://github.com/ignite/cli/pull/3860) Fix analytics event name
 
 ## [`v28.0.0`](https://github.com/ignite/cli/releases/tag/v28.0.0)
 

diff --git a/ignite/pkg/gacli/gacli.go b/ignite/pkg/gacli/gacli.go
@@ -5,6 +5,7 @@ import (
 	"encoding/json"
 	"net/http"
 	"net/url"
+	"strings"
 	"time"
 
 	"github.com/ignite/cli/v28/ignite/pkg/errors"
@@ -115,7 +116,7 @@ func (c Client) SendMetric(metric Metric) error {
 	return c.Send(Body{
 		ClientID: metric.SessionID,
 		Events: []Event{{
-			Name:   metric.Cmd,
+			Name:   strings.ReplaceAll(metric.Cmd, " ", "_"),
 			Params: metric,
 		}},
 	})