Skip to content

Code Insights: [FE] Fix insight pings event names

Administrator requested to merge vk/fix-pings-event-names into main

Created by: vovakulikov

Context

This PR fixes insight pings event names. In the 3.33 release, we've made a mistake and with ping refactoring PR we also changed event tracking names on FE (but they stayed the same on the BE https://sourcegraph.com/github.com/sourcegraph/sourcegraph/-/blob/internal/usagestats/code_insights.go?L26&subtree=true). Cause of that we lost our ping data for 3.33 customers.

@unclejustin I think it makes sense to kick off this PR again https://github.com/sourcegraph/sourcegraph/pull/24477 to avoid this problem in the future.

How to test

  1. Create/Edit/Delete insight with opened network tab.
  2. Track that these pings events have the right names.
  3. Check https://k8s.sgdev.org/site-admin/pings and track code insights latest pings have a different value (for example WeeklyInsightCreators ping

@coury-clark how often these pings on this page https://k8s.sgdev.org/site-admin/pings are sent?

Merge request reports