-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Add daily/monthly usage ping P3A metrics for iOS #36177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Would it be possible to uplift this to 1.63 @iccub NVM, looks like it is being uplifted, thanks |
@DJAndries @kylehickinson can we get some STR/Cases added for the above? Or is it the same cases as brave/brave-core#17649 (comment)? If that's the case, lets update the issue/PR to mention what needs to be checked/verified 👍 |
@kjozwiak it should be the same as desktop, the main difference is that iOS isn't implementing the wallet-related usage P3A metrics directly, just |
@DJAndries @kylehickinson do we need to update the |
The above requires |
This currently won't be verifiable on release because access to debug menus is not available there just fyi |
updated the wiki 👍 |
@kylehickinson in that case, lets just |
Please replicate #29122. Please uplift to release once complete.
cc @brave/ios
The text was updated successfully, but these errors were encountered: