Skip to content

Commit

Permalink
Remove Node Run events from GH nodes (#3674)
Browse files Browse the repository at this point in the history
Removed tracking for grasshopper node runs
  • Loading branch information
JR-Morgan authored Jan 14, 2025
1 parent 516caa6 commit e061225
Showing 1 changed file with 9 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,13 +35,15 @@ public void TrackNodeCreation(string? name = null)

public void TrackNodeRun(string? name = null, string? node = null)
{
var customProperties = new Dictionary<string, object> { { "name", name ?? Parent?.Name ?? "unset" } };
if (node != null)
{
customProperties.Add("node", node);
}
AppendHostAppInfoToProperties(customProperties);
Speckle.Core.Logging.Analytics.TrackEvent(Speckle.Core.Logging.Analytics.Events.NodeRun, customProperties);
// Node Run tracking is disabled to prevent flooding Mixpanel (see https://linear.app/speckle/issue/CNX-1042/remove-node-run-events-from-gh-nodes)

// var customProperties = new Dictionary<string, object> { { "name", name ?? Parent?.Name ?? "unset" } };
// if (node != null)
// {
// customProperties.Add("node", node);
// }
// AppendHostAppInfoToProperties(customProperties);
// Speckle.Core.Logging.Analytics.TrackEvent(Speckle.Core.Logging.Analytics.Events.NodeRun, customProperties);
}

public void TrackNodeSend(Account acc, bool auto, string? workspaceId, bool sync = false)
Expand Down

0 comments on commit e061225

Please sign in to comment.