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 custom Brave Ad notifications #9592

Closed
tmancey opened this issue May 3, 2020 · 29 comments · Fixed by brave/brave-core#6728
Closed

Implement custom Brave Ad notifications #9592

tmancey opened this issue May 3, 2020 · 29 comments · Fixed by brave/brave-core#6728

Comments

@tmancey
Copy link
Contributor

tmancey commented May 3, 2020

Implement custom Brave Ad notifications

@tmancey tmancey added design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. needs-more-info The report requires more detail before we can decide what to do with this issue. feature/ads labels May 3, 2020
@tmancey
Copy link
Contributor Author

tmancey commented May 3, 2020

Resolving this issue would close the following issues:

#5541 Fix user earns while native macOS notifications are suppressed
#5205 Clearly mark all Brave Ads as Brave Ads
#2887 Brave Ads notification icon is too small and almost invisible
#4007 Brave Ads notification uses release channel icon
#6913 Brave Ads - major UX problem
#4194 Feature request: Track what fraction of Brave Ads system notifications sent cannot be delivered, on a per-platform basis
#7867 Investigate Chromium XPC service native notification changes for macOS Catalina
#5542 Fix user earns while native Brave Ads Linux notifications are suppressed

@UtkarshVerma
Copy link

@tmancey This is how my ad notifications look like.
pic-selected-210414-2335-18

@tmancey Any thoughts?

@tmancey
Copy link
Contributor Author

tmancey commented May 21, 2021

@UtkarshVerma you cannot disable custom ad notifications. If you are experiencing issues, please raise a new github issue with steps to reproduce. Thank you

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 8, 2021

Verification passed on



<!--StartFragment-->
Brave | 1.26.54 Chromium: 91.0.4472.88&nbsp;(Official Build)&nbsp;beta&nbsp;(64-bit)
-- | --
Revision | 109e9cd038b94a631aea7d40ee3d56c1278f2597-refs/branch-heads/4472@{#1385}
OS | Windows&nbsp;10 OS Version 2004 (Build 19041.985)

<!--EndFragment-->


Verification passed on

Brave 1.26.53 Chromium: 91.0.4472.77 (Official Build) beta (64-bit)
Revision 1cecd5c8a856bc2a5adda436e7b84d8d21b339b6-refs/branch-heads/4472@{#1246}
OS Ubuntu 18.04 LTS

Verification done in #13132 (comment)


Verified PASSED using

Brave 1.26.54 Chromium: 91.0.4472.88 (Official Build) beta (x86_64)
Revision 109e9cd038b94a631aea7d40ee3d56c1278f2597-refs/branch-heads/4472@{#1385}
OS macOS Version 11.4 (Build 20F71)

Verification done in #13132 (comment)

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 14, 2021

Verification passed on Samsung Galaxy Tab version 10 running brave Beta build Bravemonoarm64 1.26.56.apk
Verified the test plan from brave/brave-core#8377

Brave://rewards - Ads panel:


Verification passed on OnePlus 6T with Android 10 running 1.26.59 x64 beta build

  • Verified custom ad notifications are shown
  • Verified View/Click/Landed events are captured correctly
  • Verified ad notification is auto-dismissed after 30 secs of no interaction
  • Verified swiping between apps when custom ad notification is shown is still retained within 30 secs
  • Verified clicking on the ad notification loads the landing page
  • Verified users are rewarded for viewing custom ads

View Event

2021-06-14 10:14:43.870 27076-27076/? V/chromium: [VERBOSE1:ad_notification_serving.cc(105)] Ad notification delivered:
      uuid: 03022852-d38a-493a-8bb0-47ffcfbf9704
      creativeInstanceId: ed6c467a-8643-42eb-a7cd-32e878fa3cd4
      creativeSetId: a9f0b49c-3298-4c1b-a8b5-76a71e8d7332
      campaignId: 0e70c893-b0c0-4dab-860d-7e65446f0f0f
      advertiserId: 8d2f4191-b76e-4369-8983-3d9ff551c5ad
      segment: untargeted
      title: This is a test
      body: This is a test
      targetUrl: https://www.brave.com
2021-06-14 10:14:43.906 27076-27076/? V/chromium: [VERBOSE3:ad_notification_event_viewed.cc(24)] Viewed ad notification with uuid 03022852-d38a-493a-8bb0-47ffcfbf9704 and creative instance id ed6c467a-8643-42eb-a7cd-32e878fa3cd4

Click Event

2021-06-14 10:14:53.988 27076-27076/? V/chromium: [VERBOSE6:redeem_unblinded_token.cc(84)] URL Response:
      URL: https://ads-serve.bravesoftware.com/v1/confirmation/3b55467f-9299-4555-8c1a-aadcfc017970/eyJwYXlsb2FkIjoie1wiYmxpbmRlZFBheW1lbnRUb2tlblwiOlwiMXIxRHVGRWMrcjAwY3o0SWJHYkJSUmEvallHaWozcS9EVG5PdzRNZ1ZpST1cIixcImJ1aWxkQ2hhbm5lbFwiOlwiYmV0YVwiLFwiY3JlYXRpdmVJbnN0YW5jZUlkXCI6XCJlZDZjNDY3YS04NjQzLTQyZWItYTdjZC0zMmU4NzhmYTNjZDRcIixcInBheWxvYWRcIjp7fSxcInBsYXRmb3JtXCI6XCJhbmRyb2lkXCIsXCJzdHVkaWVzXCI6W10sXCJ0eXBlXCI6XCJjbGlja1wifSIsInNpZ25hdHVyZSI6ImQrakdObGk3VVRPN0Q0QzlpMUZuZTFIeHA4Vnh3OWFjTUdmaWk0SEd1WHU0SnJ3RldIY2Qzd3gvbHhnY1QrbUZoZE1WMEZ5dFl0dWN0Z2dWZHhLNUtRPT0iLCJ0IjoiNndxSHNQRGEvSjEvajJybHV2OHlneHdqR3NjZDdmYW85L2habUQ3SkZxRXpUT2FkZlplR0RFZ3FYd1JZL0VwNFVNQzB5UzZERm8ralowU0hTZzRPK2c9PSJ9
      Response Status Code: 201
      Response: {"id":"3b55467f-9299-4555-8c1a-aadcfc017970","payload":{},"createdAt":"2021-06-14T14:14:53.832Z","type":"click","modifiedAt":"2021-06-14T14:14:53.832Z","creativeInstanceId":"ed6c467a-8643-42eb-a7cd-32e878fa3cd4"}

Landed Event

2021-06-14 10:15:07.246 27076-27076/? V/chromium: [VERBOSE6:redeem_unblinded_token.cc(84)] URL Response:
      URL: https://ads-serve.bravesoftware.com/v1/confirmation/d7bc4109-3864-43dd-a74c-e2bbc026504b/eyJwYXlsb2FkIjoie1wiYmxpbmRlZFBheW1lbnRUb2tlblwiOlwiTU1pWUorWExtWHM5K0FseFpVVTJRQ1JCWWhsNjlPOE8yank4MW9McktFZz1cIixcImJ1aWxkQ2hhbm5lbFwiOlwiYmV0YVwiLFwiY3JlYXRpdmVJbnN0YW5jZUlkXCI6XCJlZDZjNDY3YS04NjQzLTQyZWItYTdjZC0zMmU4NzhmYTNjZDRcIixcInBheWxvYWRcIjp7fSxcInBsYXRmb3JtXCI6XCJhbmRyb2lkXCIsXCJzdHVkaWVzXCI6W10sXCJ0eXBlXCI6XCJsYW5kZWRcIn0iLCJzaWduYXR1cmUiOiJkbmR5Q3UzQ0J3UGdGbHlVWkFjWVpsWC9qSVhFd2RuUW5zUFdDNFZ1WnJkbjZlazdSZ0tiNkFKcTdSMTBNNlNLYTNZcVpKMzhSM01LMUM4dGl1ZGVyQT09IiwidCI6InFWYzl5R25jczV1VHZIRDl2UEZrZ2FWWDlacUVQQW5zakppeUpkYmZPclJrYmlHZ091a2NEdkN6LzVJaXZycDRDRDQwUHR0RDc1RGRFRDVSSkQ5aXpBPT0ifQ==
      Response Status Code: 201
      Response: {"id":"d7bc4109-3864-43dd-a74c-e2bbc026504b","payload":{},"createdAt":"2021-06-14T14:15:06.928Z","type":"landed","modifiedAt":"2021-06-14T14:15:06.928Z","creativeInstanceId":"ed6c467a-8643-42eb-a7cd-32e878fa3cd4"}

Dismiss event

@CodeDead
Copy link

Notifications are not working on the Wayland display server.

Steps to reproduce:

  • Use Linux
  • Use Wayland
  • Use Brave

@mattfbacon
Copy link

PLEASE add an option to disable this. I hate these notifications, they display on top of actual notifications, they are ugly compared to my actual notifications. Everything was working fine before.

@tmancey
Copy link
Contributor Author

tmancey commented Oct 8, 2021

Hi. You can disable custom ad notifications in brave://flags. Thanks

@mattfbacon
Copy link

Is that "Enable system notifications"? Or some other key?

@mattfbacon
Copy link

Nevermind, it's "Enable Brave Ads custom notifications". Thanks. Hope this helps anyone that wants system notifications.

@mattfbacon
Copy link

Actually, that doesn't work. Could you give the specific flag to set?

@UndarkAido
Copy link

UndarkAido commented Oct 17, 2021

brave://flags/#brave-ads-custom-notifications isn't working for me either. I'm just going to turn off ads for now.

@UndarkAido
Copy link

Looks like #18798 covers this issue though for anyone coming through

@tmancey
Copy link
Contributor Author

tmancey commented Oct 18, 2021

Please try again as you should now see native notifications on the release build

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