-
Notifications
You must be signed in to change notification settings - Fork 1.3k
For #18854: Add metrics for home screen being displayed. #19025
Conversation
Request for data collection review formAll questions are mandatory. You must receive review from a data steward peer on your responses to these questions before shipping new data collection.
|
tagging @tdsmith for data review. |
Codecov Report
@@ Coverage Diff @@
## master #19025 +/- ##
============================================
- Coverage 33.67% 33.64% -0.03%
+ Complexity 1545 1544 -1
============================================
Files 532 532
Lines 21591 21595 +4
Branches 3218 3220 +2
============================================
- Hits 7270 7265 -5
- Misses 13433 13446 +13
+ Partials 888 884 -4
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
This pull request has conflicts when rebasing. Could you fix it @mcarare? 🙏 |
data-review+, thanks!
Yes, in the probe definition files and the Glean probe dictionary.
Yes, the Firefox telemetry opt-out.
n/a
Cat 2, interaction.
Default-on.
No.
Yes.
The team is responsible for renewing as needed.
No. |
@tdsmith Thank you! |
test-debug seems stuck. Closing and reopening. |
Pull Request checklist
To download an APK when reviewing a PR: