-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: even more taxonomy fangling #25650
Conversation
pauldambra
commented
Oct 17, 2024
- adds some missed billing properties
- moves a few non dollar properties out since some events are pretty bare without anything shown
- adds another surveys event to the taxonomy so it has nice display in the table
@@ -1212,6 +1212,10 @@ export const CORE_FILTER_DEFINITIONS_BY_GROUP = { | |||
description: 'posthog-js adds these to the page leave event, they are used in web analytics calculations', | |||
label: 'Previous pageview duration', | |||
}, | |||
$surveys_activated: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Phanatic I took a guess at a description here, feel free to correct :)
(at the risk of teaching you to suck eggs... adding event properties here means when we display them we add a logo at the front and show the description in a pop over so folk know they're legit PH properties)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no idea how to eat eggs, I just end up scrambling them :)
Good callout on the event properties! Sounds good to me.
Size Change: +109 B (+0.01%) Total Size: 1.14 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ship it!