-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Instrumentation Feb-8 #3238
Instrumentation Feb-8 #3238
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.
Some comments, but otherwise almost ready!
81b4b78
to
770fa37
Compare
Thanks for the review @macobo 💥 ! Feedback addressed and PR rebased. Will merge after tests pass. For future reference, cloud tests will fail because we introduced more attributes to the signup (which is overriden on posthog-production). Will fix the tests in posthog-production post-merge. |
Changes
This PR introduces more instrumentation that will help improve our product.
event_usage.py
module to do the same function aseventUsageLogic.ts
in the BE. Related to that, abstracted some event usage reporting logic here.$identify
related to when the user's account was actually created and SSO usage.Checklist