Sulkta-OSS/element-x-ada
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Init or stop posthog based on user consent

Browse source
This commit is contained in:
yostyle 2023-06-23 15:57:25 +02:00
parent fb23350b62
commit 09c2c3dea1
3 changed files with 10 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

7
services/analytics/impl/src/main/kotlin/io/element/android/services/analytics/impl/DefaultAnalyticsService.kt
View file

@ -115,15 +115,16 @@ class DefaultAnalyticsService @Inject constructor(
} }
private fun initOrStop() { private fun initOrStop() {
userConsent?.let { _userConsent -> userConsent?.let { userConsent ->
when (_userConsent) { when (userConsent) {
true -> { true -> {
analyticsProviders.onEach { it.init() }
pendingUserProperties?.let { pendingUserProperties?.let {
analyticsProviders.onEach { provider -> provider.updateUserProperties(it) } analyticsProviders.onEach { provider -> provider.updateUserProperties(it) }
pendingUserProperties = null pendingUserProperties = null
} }
} }
false -> {} false -> analyticsProviders.onEach { it.stop() }
} }
} }
} }

2
services/analyticsproviders/api/src/main/kotlin/io/element/android/services/analyticsproviders/api/AnalyticsProvider.kt
View file

@ -30,7 +30,7 @@ interface AnalyticsProvider: AnalyticsTracker, ErrorTracker {
*/ */
val name: String val name: String
suspend fun init() fun init()
fun stop() fun stop()
} }

10
services/analyticsproviders/posthog/src/main/kotlin/io/element/android/services/analyticsproviders/posthog/PosthogAnalyticsProvider.kt
View file

@ -42,7 +42,7 @@ class PosthogAnalyticsProvider @Inject constructor(
private var posthog: PostHog? = null private var posthog: PostHog? = null
private var analyticsId: String? = null private var analyticsId: String? = null
override suspend fun init() { override fun init() {
posthog = createPosthog() posthog = createPosthog()
posthog?.optOut(false) posthog?.optOut(false)
identifyPostHog() identifyPostHog()
@ -66,10 +66,10 @@ class PosthogAnalyticsProvider @Inject constructor(
} }
override fun updateUserProperties(userProperties: UserProperties) { override fun updateUserProperties(userProperties: UserProperties) {
posthog?.identify( // posthog?.identify(
REUSE_EXISTING_ID, userProperties.getProperties()?.toPostHogUserProperties(), // REUSE_EXISTING_ID, userProperties.getProperties()?.toPostHogUserProperties(),
IGNORED_OPTIONS // IGNORED_OPTIONS
) // )
} }
override fun trackError(throwable: Throwable) { override fun trackError(throwable: Throwable) {