-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Implement Ads tab on brave://rewards-internals with diagnostic information #10964
Comments
I've added the following:
|
Following diagnostics added: - Users locale [LocaleHelper] - Status: Enabled/disabled [prefs::kEnabled] - Catalog id [prefs::kCatalogId] - Catalog last updated timestamp [prefs::kCatalogLastUpdated] - Last un-idle timestamp [AdsImpl::OnUnIdle] fix brave/brave-browser#10964
Following diagnostics added: - Users locale [LocaleHelper] - Status: Enabled/disabled [prefs::kEnabled] - Catalog id [prefs::kCatalogId] - Catalog last updated timestamp [prefs::kCatalogLastUpdated] - Last un-idle timestamp [AdsImpl::OnUnIdle] fix brave/brave-browser#10964
MVP delivered with
We will write up a document for sign-off from the support and privacy teams for additional data points |
Following diagnostics added: - Users locale [LocaleHelper] - Status: Enabled/disabled [prefs::kEnabled] - Catalog id [prefs::kCatalogId] - Catalog last updated timestamp [prefs::kCatalogLastUpdated] - Last un-idle timestamp [AdsImpl::OnUnIdle] fix brave/brave-browser#10964 # Conflicts: # vendor/bat-native-ads/include/bat/ads/ads.h
Following diagnostics added: - Users locale [LocaleHelper] - Status: Enabled/disabled [prefs::kEnabled] - Catalog id [prefs::kCatalogId] - Catalog last updated timestamp [prefs::kCatalogLastUpdated] - Last un-idle timestamp [AdsImpl::OnUnIdle] fix brave/brave-browser#10964 # Conflicts: # vendor/bat-native-ads/include/bat/ads/ads.h
Verified
Steps:
Verified
Steps:
Verification passed on
Verified the test plan from brave/brave-core#9553 Clean profile Enable rewards/ads via BR panel:
Enable rewards/ads via brave rewards side bar option:
Enable rewards/ads via brave rewards settings page:
Enable rewards, disable Ads
Upgrade profile: Install 1.29.x and enable rewards and upgrade profile to 1.30.x check Ad diagnostic stats
Install 1.29.x and enable rewards, disable ads and upgrade profile to 1.30.x check Ad diagnostic stats
|
@stephendonner would you mind raising an issue for ads as if disabled we should show |
Filed #17535. |
Quick verification, and it'd be good to have this be tested on all platforms, being ads-related 👍 |
Enabled
/Disabled
en_US
9b545656-f314-48d7-9c52-978f68491b35
06/27/21 05:43:12 PM
06/27/21 05:43:12 PM
The text was updated successfully, but these errors were encountered: