diff --git a/app/metrics.yaml b/app/metrics.yaml index 9d52cd572..2a6f09251 100644 --- a/app/metrics.yaml +++ b/app/metrics.yaml @@ -3896,40 +3896,6 @@ top_sites: - android-probes@mozilla.com expires: "2022-02-01" -about_page: - support_tapped: - type: event - description: | - A user tapped on "Support" item from About page - bugs: - - https://github.com/mozilla-mobile/fenix/issues/6834 - data_reviews: - - https://github.com/mozilla-mobile/fenix/pull/8047 - - https://github.com/mozilla-mobile/fenix/pull/13958#issuecomment-676857877 - - https://github.com/mozilla-mobile/fenix/pull/18143 - - https://github.com/mozilla-mobile/fenix/pull/19924#issuecomment-861423789 - data_sensitivity: - - interaction - notification_emails: - - android-probes@mozilla.com - expires: "2021-07-01" - privacy_notice_tapped: - type: event - description: | - A user tapped on "Privacy notice" item from About page - bugs: - - https://github.com/mozilla-mobile/fenix/issues/6834 - data_reviews: - - https://github.com/mozilla-mobile/fenix/pull/8047 - - https://github.com/mozilla-mobile/fenix/pull/13958#issuecomment-676857877 - - https://github.com/mozilla-mobile/fenix/pull/18143 - - https://github.com/mozilla-mobile/fenix/pull/19924#issuecomment-861423789 - data_sensitivity: - - interaction - notification_emails: - - android-probes@mozilla.com - expires: "2021-07-01" - app_theme: dark_theme_selected: type: event diff --git a/app/src/main/java/org/mozilla/fenix/components/metrics/Event.kt b/app/src/main/java/org/mozilla/fenix/components/metrics/Event.kt index 023619f7f..ba08e370c 100644 --- a/app/src/main/java/org/mozilla/fenix/components/metrics/Event.kt +++ b/app/src/main/java/org/mozilla/fenix/components/metrics/Event.kt @@ -141,8 +141,6 @@ sealed class Event { object CustomEngineDeleted : Event() object PrivateBrowsingShowSearchSuggestions : Event() object WhatsNewTapped : Event() - object SupportTapped : Event() - object PrivacyNoticeTapped : Event() object PocketTopSiteClicked : Event() object PocketTopSiteRemoved : Event() object FennecToFenixMigrated : Event() diff --git a/app/src/main/java/org/mozilla/fenix/components/metrics/GleanMetricsService.kt b/app/src/main/java/org/mozilla/fenix/components/metrics/GleanMetricsService.kt index ce762f8f8..e266c9bd1 100644 --- a/app/src/main/java/org/mozilla/fenix/components/metrics/GleanMetricsService.kt +++ b/app/src/main/java/org/mozilla/fenix/components/metrics/GleanMetricsService.kt @@ -8,7 +8,6 @@ import android.content.Context import mozilla.components.service.glean.Glean import mozilla.components.service.glean.private.NoExtraKeys import mozilla.components.support.base.log.logger.Logger -import org.mozilla.fenix.GleanMetrics.AboutPage import org.mozilla.fenix.GleanMetrics.Addons import org.mozilla.fenix.GleanMetrics.AndroidAutofill import org.mozilla.fenix.GleanMetrics.AndroidKeystoreExperiment @@ -534,12 +533,6 @@ private val Event.wrapper: EventWrapper<*>? { TopSites.swipeCarousel.record(it) }, { TopSites.swipeCarouselKeys.valueOf(it) } ) - is Event.SupportTapped -> EventWrapper( - { AboutPage.supportTapped.record(it) } - ) - is Event.PrivacyNoticeTapped -> EventWrapper( - { AboutPage.privacyNoticeTapped.record(it) } - ) is Event.PocketTopSiteClicked -> EventWrapper( { Pocket.pocketTopSiteClicked.record(it) } ) diff --git a/app/src/main/java/org/mozilla/fenix/settings/about/AboutFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/about/AboutFragment.kt index 6adc1f03f..34281e713 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/about/AboutFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/about/AboutFragment.kt @@ -198,13 +198,7 @@ class AboutFragment : Fragment(), AboutPageListener { WhatsNew.userViewedWhatsNew(requireContext()) requireComponents.analytics.metrics.track(Event.WhatsNewTapped) } - SUPPORT -> { - requireComponents.analytics.metrics.track(Event.SupportTapped) - } - PRIVACY_NOTICE -> { - requireComponents.analytics.metrics.track(Event.PrivacyNoticeTapped) - } - LICENSING_INFO, RIGHTS -> {} // no telemetry needed + SUPPORT, PRIVACY_NOTICE, LICENSING_INFO, RIGHTS -> {} // no telemetry needed } openLinkInNormalTab(item.url)