Bug 1853707 - Send Search Widget telemetry when primary button tapped

Send the add search widget click telemetry when the primary
button has been clicked on the juno onboarding card.

(cherry picked from commit d5965e4427d3646c9cdece1e2b79cd654b9399d8)
fenix/118.1.1^2
DreVla 8 months ago committed by Mergify
parent d752220a43
commit 1076583547

@ -157,6 +157,10 @@ class JunoOnboardingFragment : Fragment() {
)
},
onAddFirefoxWidgetClick = {
telemetryRecorder.onAddSearchWidgetClick(
pagesToDisplay.telemetrySequenceId(),
pagesToDisplay.sequencePosition(OnboardingPageUiData.Type.ADD_SEARCH_WIDGET),
)
showAddSearchWidgetDialog()
},
onSkipFirefoxWidgetClick = {

@ -39,7 +39,6 @@ import org.mozilla.fenix.R
import org.mozilla.fenix.components.components
import org.mozilla.fenix.compose.PagerIndicator
import org.mozilla.fenix.compose.annotation.LightDarkPreview
import org.mozilla.fenix.onboarding.JunoOnboardingTelemetryRecorder
import org.mozilla.fenix.onboarding.WidgetPinnedReceiver.WidgetPinnedState
import org.mozilla.fenix.theme.FirefoxTheme
@ -80,7 +79,6 @@ fun JunoOnboardingScreen(
val pagerState = rememberPagerState()
val isSignedIn: State<Boolean?> = components.backgroundServices.syncStore
.observeAsComposableState { it.account != null }
val telemetryRecorder by lazy { JunoOnboardingTelemetryRecorder() }
val widgetPinnedFlow: StateFlow<Boolean> = WidgetPinnedState.isPinned
val isWidgetPinnedState by widgetPinnedFlow.collectAsState()
@ -115,10 +113,6 @@ fun JunoOnboardingScreen(
LaunchedEffect(isWidgetPinnedState) {
if (isWidgetPinnedState) {
scrollToNextPageOrDismiss()
telemetryRecorder.onAddSearchWidgetClick(
pagesToDisplay.telemetrySequenceId(),
pagesToDisplay.sequencePosition(OnboardingPageUiData.Type.ADD_SEARCH_WIDGET),
)
}
}

Loading…
Cancel
Save