-
Notifications
You must be signed in to change notification settings - Fork 102
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
Bump posthog SDK version to 3.2.0 #2788
Conversation
Generated by 🚫 Danger Swift against e1499cb |
ElementX/Sources/Services/Analytics/PostHogAnalyticsClient.swift
Outdated
Show resolved
Hide resolved
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #2788 +/- ##
===========================================
+ Coverage 73.32% 73.35% +0.03%
===========================================
Files 597 597
Lines 42686 42696 +10
===========================================
+ Hits 31298 31320 +22
+ Misses 11388 11376 -12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
Bump posthog version to 3.2.0.
We were using a quite old posthog sdk. Updating to the new one mainly because it now support sessions. Allow us to have a per device view instead of only unique users that is merging events from all the devices of the same user.
Element X Android is already using posthog 3.x
Pull Request Checklist
UI changes have been tested with: