Analytics: inject analyticsTracker, it has a better scope
This commit is contained in:
parent
880b97cc5c
commit
c0aa0cef62
|
@ -49,7 +49,7 @@ private const val CHECK_INTERVAL = 2_000L
|
||||||
*/
|
*/
|
||||||
@Singleton
|
@Singleton
|
||||||
class DecryptionFailureTracker @Inject constructor(
|
class DecryptionFailureTracker @Inject constructor(
|
||||||
private val vectorAnalytics: VectorAnalytics,
|
private val analyticsTracker: AnalyticsTracker,
|
||||||
private val clock: Clock
|
private val clock: Clock
|
||||||
) {
|
) {
|
||||||
|
|
||||||
|
@ -136,7 +136,7 @@ class DecryptionFailureTracker @Inject constructor(
|
||||||
// for now we ignore events already reported even if displayed again?
|
// for now we ignore events already reported even if displayed again?
|
||||||
.filter { alreadyReported.contains(it).not() }
|
.filter { alreadyReported.contains(it).not() }
|
||||||
.forEach { failedEventId ->
|
.forEach { failedEventId ->
|
||||||
vectorAnalytics.capture(Error(failedEventId, Error.Domain.E2EE, aggregation.key))
|
analyticsTracker.capture(Error(failedEventId, Error.Domain.E2EE, aggregation.key))
|
||||||
alreadyReported.add(failedEventId)
|
alreadyReported.add(failedEventId)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue