mozilla-mobile / firefox-ios

Firefox for iOS
Mozilla Public License 2.0
12.08k stars 2.88k forks source link

Telemetry - notifications #14907

Open data-sync-user opened 1 year ago

data-sync-user commented 1 year ago

┆Issue is synchronized with this Jira Task

data-sync-user commented 1 year ago

➤ Nishant Patel commented:

PLEASE NOTE that the metric names “sync_notifications_allowed“ and “tips_and_features_notifications_allowed“ have been shortened to sync_notifs and tips_and_features_notifs because the YAML rejects long names.

Daniela Arcese

data-sync-user commented 1 year ago

➤ Nishant Patel commented:

ALSO NOTE the change value for preferences.changed is already in place, https://github.com/mozilla-mobile/firefox-ios/blob/d81c39a86db60875c0e956563bb0e7a85b7ac95c/Client/metrics.yaml#L1941 ( https://github.com/mozilla-mobile/firefox-ios/blob/d81c39a86db60875c0e956563bb0e7a85b7ac95c/Client/metrics.yaml#L1941|smart-link ) . We are already sending the preference key and the value it changed to.

data-sync-user commented 1 year ago

➤ Daniela Arcese commented:

Nishant Patel please put a comment in the spreadsheet with the implemented names

github-actions[bot] commented 4 weeks ago

This issue has been automatically marked as stale. Has the issue been fixed, or does it still require the community's attention? Please leave any comment to keep this issue opened. It will be closed automatically if no further update occurs in the next 30 days. Thank you for your contributions!