-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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: update taxomony with new property #28339
Conversation
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.
PR Summary
Added $last_posthog_reset
property to the taxonomy files to track the timestamp of the last web SDK Reset call, aligning documentation with functionality introduced in version 1.207.5.
- Added
$last_posthog_reset
toevent_properties
in/frontend/src/lib/taxonomy.tsx
with debugging description - Added corresponding property definition in
/posthog/taxonomy/taxonomy.py
marked as system property - Property marked to be ignored by AI assistant to avoid unnecessary surfacing in recommendations
2 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings | Greptile
posthog/taxonomy/taxonomy.py
Outdated
@@ -272,6 +272,12 @@ class CoreFilterDefinition(TypedDict): | |||
}, | |||
}, | |||
"event_properties": { | |||
"$last_posthog_reset": { | |||
"label": "Timestamp of last call to `Reset`in the web sdk", |
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.
syntax: Missing space between Reset
and in
in the label text
frontend/src/lib/taxonomy.tsx
Outdated
label: 'Timestamp of last call to `Reset`in the web sdk', | ||
description: 'The timestamp of the last call to `Reset` in the web SDK. This can be useful for debugging.', |
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.
syntax: typo in 'Reset`in' - missing space
Size Change: +195 B (+0.02%) Total Size: 1.17 MB ℹ️ View Unchanged
|
we added this in 1.207.5
but forgot to update the taxonomy