Bug 1875514 - Only send installation ping when we receive adjust metrics

fenix/124.1.0
Roger Yang 4 months ago committed by mergify[bot]
parent bef8bc3749
commit 7dc1b784b4

@ -843,6 +843,17 @@ events:
metadata:
tags:
- Search
first_session_ping_cancelled:
type: event
description: |
First session ping cancelled because Adjust metrics were empty.
bugs:
- https://bugzilla.mozilla.org/show_bug.cgi?id=1875514
data_reviews:
- https://github.com/mozilla-mobile/firefox-android/pull/5223
notification_emails:
- android-probes@mozilla.com
expires: never
splash_screen:
first_launch_extended:

@ -10,8 +10,10 @@ import androidx.annotation.VisibleForTesting
import kotlinx.coroutines.CoroutineScope
import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.launch
import mozilla.components.service.glean.private.NoExtras
import mozilla.components.support.base.log.logger.Logger
import org.mozilla.fenix.Config
import org.mozilla.fenix.GleanMetrics.Events
import org.mozilla.fenix.GleanMetrics.FirstSession
import org.mozilla.fenix.GleanMetrics.Pings
import org.mozilla.fenix.ext.settings
@ -74,6 +76,8 @@ class FirstSessionPing(private val context: Context) {
Pings.firstSession.submit()
markAsTriggered()
}
} else {
Events.firstSessionPingCancelled.record(NoExtras())
}
}

@ -66,7 +66,6 @@ class GleanMetricsService(
private var initialized = false
private val activationPing = ActivationPing(context)
private val installationPing = FirstSessionPing(context)
override fun start() {
logger.debug("Enabling Glean.")
@ -87,7 +86,6 @@ class GleanMetricsService(
Glean.registerPings(Pings)
activationPing.checkAndSend()
installationPing.checkAndSend()
}
}

Loading…
Cancel
Save