Skip to content
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

Closed
tmancey opened this issue Jul 28, 2020 · 7 comments · Fixed by brave/brave-core#9553
Closed

Comments

@tmancey
Copy link
Contributor

tmancey commented Jul 28, 2020

  • Status: Enabled / Disabled
  • Users locale en_US
  • Catalog id 9b545656-f314-48d7-9c52-978f68491b35
  • Catalog last updated timestamp 06/27/21 05:43:12 PM
  • Last un-idle timestamp 06/27/21 05:43:12 PM
@tmancey tmancey changed the title [Desktop] Implement Ads tab on brave://rewards-internals with catalog meta data Implement Ads tab on brave://rewards-internals with catalog meta data Aug 5, 2020
@tmancey tmancey added OS/Android Fixes related to Android browser functionality dev-concern and removed OS/Android Fixes related to Android browser functionality OS/Desktop labels Aug 5, 2020
@tmancey tmancey added OS/Android Fixes related to Android browser functionality OS/Desktop labels Aug 20, 2020
@tmancey tmancey changed the title Implement Ads tab on brave://rewards-internals with catalog meta data Implement Ads tab on brave://rewards-internals with diagnostic information Oct 21, 2020
@moritzhaller
Copy link

I've added the following:

- language code and page classifier model ID
- country code and purchase intent model ID

@tmancey tmancey added the priority/P3 The next thing for us to work on. It'll ride the trains. label Nov 9, 2020
@aseren aseren self-assigned this Jul 14, 2021
aseren added a commit to brave/brave-core that referenced this issue Jul 26, 2021
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
aseren added a commit to brave/brave-core that referenced this issue Aug 4, 2021
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
@tmancey
Copy link
Contributor Author

tmancey commented Aug 6, 2021

MVP delivered with

Users locale [LocaleHelper]
Status: Enabled/disabled [prefs::kEnabled]
Catalog id [prefs::kCatalogId]
Catalog last updated timestamp [prefs::kCatalogLastUpdated]
Last un-idle timestamp [AdsImpl::OnUnIdle]

We will write up a document for sign-off from the support and privacy teams for additional data points

aseren added a commit to brave/brave-core that referenced this issue Aug 9, 2021
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
aseren added a commit to brave/brave-core that referenced this issue Aug 12, 2021
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
@tmancey tmancey added this to the 1.30.x - Nightly milestone Aug 12, 2021
@stephendonner
Copy link

stephendonner commented Aug 17, 2021

Verified PASSED using

Brave 1.30.24 Chromium: 92.0.4515.131 (Official Build) nightly (x86_64)
Revision 6b8d6c56ce21e38a72f7c4becb5abc1fa5134f29-refs/branch-heads/4515@{#1933}
OS macOS Version 12.0 (Build 21A5294g)

Steps:

  1. new profile
  2. launched Brave
  3. loaded brave://rewards-internals and clicked on the Ad diagnostics tab
  4. clicked Refresh; confirmed nothing displayed
  5. opened brave://rewards and enabled Ads
  6. clicked Refresh; confirmed I got diagnostic info
  7. disabled Ads and refreshed brave://rewards-internals; confirmed I saw Ads not initialized, please refresh the page.
example example example
Screen Shot 2021-08-16 at 5 40 53 PM Screen Shot 2021-08-16 at 5 41 07 PM Screen Shot 2021-08-16 at 5 39 49 PM

Verified PASSED using

Brave 1.30.66 Chromium: 93.0.4577.63 (Official Build) dev (64-bit)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS Linux

Steps:

  1. new profile
  2. launched Brave
  3. loaded brave://rewards-internals and clicked on the Ad diagnostics tab
  4. clicked Refresh; confirmed nothing displayed
  5. opened brave://rewards and enabled Ads
  6. clicked Refresh; confirmed I got diagnostic info
  7. disabled Ads and refreshed brave://rewards-internals; confirmed I saw Ads not initialized, please refresh the page.
example example example
Screen Shot 2021-09-07 at 11 26 40 AM Screen Shot 2021-09-07 at 11 27 21 AM Screen Shot 2021-09-07 at 11 27 25 AM

Verification passed on

Brave | 1.30.65 Chromium: 93.0.4577.63 (Official Build) beta (64-bit)
-- | --
Revision | ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS | Windows 10 OS Version 2009 (Build 19043.1165)

Verified the test plan from brave/brave-core#9553

Clean profile

Enable rewards/ads via BR panel:

rewards/ads off rewards/ads on
image image

Enable rewards/ads via brave rewards side bar option:

rewards/ads off rewards/ads on
image image

Enable rewards/ads via brave rewards settings page:

rewards/ads off rewards/ads on
image image

Enable rewards, disable Ads

rewards/ads off rewards/ads on
image image

Upgrade profile:

Install 1.29.x and enable rewards and upgrade profile to 1.30.x check Ad diagnostic stats

1.29.x 1.30.x
image image

Install 1.29.x and enable rewards, disable ads and upgrade profile to 1.30.x check Ad diagnostic stats

1.29.x 1.30.x
image image

@tmancey
Copy link
Contributor Author

tmancey commented Aug 17, 2021

@stephendonner would you mind raising an issue for ads as if disabled we should show Ads enabled: false. Thanks

@stephendonner
Copy link

@stephendonner would you mind raising an issue for ads as if disabled we should show Ads enabled: false. Thanks

Filed #17535.

@stephendonner
Copy link

Quick verification, and it'd be good to have this be tested on all platforms, being ads-related 👍

@srirambv
Copy link
Contributor

Verification passed on Oppo Reno 5 with Android 11 running 1.30.76 x64 build

  • Verified when ads is disabled, diagnostics page shows ads not initialized
  • Verified when ads is enabled, diagnostics page shows details about Location/Catalog ID/Last updated datetime stamp
image image

Verification passed on Samsung Tab A with Android 10 running 1.30.76 x64 build

  • Verified when ads is disabled, diagnostics page shows ads not initialized
  • Verified when ads is enabled, diagnostics page shows details about Location/Catalog ID/Last updated datetime stamp
image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment