Skip to content

Conversation

@latekvo
Copy link
Member

@latekvo latekvo commented Dec 17, 2025

The editor type is currently embedded inside the event name. That complicates data analysis.

This PR fixes this issue, and adds data on the active license type (Free, Pro, Inactive, and others).

Note: As far as I see, there isn't a way to select eventA & eventB, split by eventB without using KQL, but you can select eventA & eventB, split by eventB.myProperty, which is why we're using props.

How Has This Been Tested:

  • Checked input properties at the time of sendTelemetryEvent call
  • All properties match the expected value

How Has This Change Been Documented:

Not applicable

@vercel

This comment was marked as spam.

This comment was marked as spam.

Copy link
Contributor

@jwajgelt jwajgelt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool stuff.
We didn't do this previously because the folks working with telemetry preferred separate events, rather than params -- but since you're the one looking at this, I assume you've figured it out 👍

@latekvo latekvo merged commit f97af3d into main Dec 19, 2025
9 checks passed
@latekvo latekvo deleted the @latekvo/telemetry-for-license-type branch December 19, 2025 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants