From 980b11610ba13949087a7e71f2f4fe6ec67ab3ad Mon Sep 17 00:00:00 2001 From: Sawyer Blatz Date: Wed, 9 Oct 2019 09:38:35 -0700 Subject: [PATCH] For #5094: Adds telemetry for show search shortcuts --- app/metrics.yaml | 3 ++- .../main/java/org/mozilla/fenix/components/metrics/Metrics.kt | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/app/metrics.yaml b/app/metrics.yaml index c31ea039d..6e355f0a0 100644 --- a/app/metrics.yaml +++ b/app/metrics.yaml @@ -111,7 +111,7 @@ events: preference_key: description: "The preference key for the switch preference the user toggled. We currently track: show_search_suggestions, remote_debugging, telemetry, tracking_protection, search_bookmarks, - search_browsing_history, and show_clipboard_suggestions" + search_browsing_history, show_clipboard_suggestions, and show_search_shortcuts" enabled: description: "Whether or not the preference is *now* enabled" bugs: @@ -120,6 +120,7 @@ events: data_reviews: - https://github.com/mozilla-mobile/fenix/pull/1896 - https://github.com/mozilla-mobile/fenix/pull/5704 + - https://github.com/mozilla-mobile/fenix/pull/5886 notification_emails: - fenix-core@mozilla.com expires: "2020-03-01" diff --git a/app/src/main/java/org/mozilla/fenix/components/metrics/Metrics.kt b/app/src/main/java/org/mozilla/fenix/components/metrics/Metrics.kt index f406f1060..d5cf0f200 100644 --- a/app/src/main/java/org/mozilla/fenix/components/metrics/Metrics.kt +++ b/app/src/main/java/org/mozilla/fenix/components/metrics/Metrics.kt @@ -137,7 +137,8 @@ sealed class Event { context.getString(R.string.pref_key_tracking_protection), context.getString(R.string.pref_key_search_bookmarks), context.getString(R.string.pref_key_search_browsing_history), - context.getString(R.string.pref_key_show_clipboard_suggestions) + context.getString(R.string.pref_key_show_clipboard_suggestions), + context.getString(R.string.pref_key_show_search_shortcuts) ) override val extras: Map?