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

Unable to unblock ads through ads history after 30 days #17713

Closed
btlechowski opened this issue Aug 26, 2021 · 4 comments
Closed

Unable to unblock ads through ads history after 30 days #17713

btlechowski opened this issue Aug 26, 2021 · 4 comments
Assignees
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. design A design change, especially one which needs input from the design team enhancement feature/ads feature/rewards needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. OS/Desktop

Comments

@btlechowski
Copy link

Follow up to #17305
The blocked ads are blocked permanently after 30 days

Important: If user had ever blocked an ad pre 1.29.x, the block would be applied in 1.29.x. This way user can be surprised that ads are no longer shown.

Steps to Reproduce

Test case 1:

  1. Follow Released notification ad after 30 days - Untargeted segment from https://github.com/brave/internal/issues/794

Test case 2:

  1. Follow Released notification ad after 30 days after upgrade - Untargeted segment from https://github.com/brave/internal/issues/794

Actual result:

Unable to unblock ads through ads history
The blocked ads are blocked permanently

ads history is empty
image

Expected result:

Able to unblock ads after 30 days

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 1.29.67 Chromium: 92.0.4515.159 (Official Build) beta (64-bit)
Revision 0185b8a19c88c5dfd3e6c0da6686d799e9bc3b52-refs/branch-heads/4515@{#2052}
OS Linux

cc @tmancey @jsecretan @moritzhaller @rebron @brave/legacy_qa

@tmancey tmancey added design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. blocked enhancement and removed bug labels Aug 27, 2021
@zenparsing zenparsing added the priority/P3 The next thing for us to work on. It'll ride the trains. label Sep 29, 2021
@tmancey tmancey added design A design change, especially one which needs input from the design team and removed priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes QA/Test-Plan-Specified design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. labels Feb 15, 2022
@tmancey
Copy link
Contributor

tmancey commented Jun 7, 2022

@btlechowski @jsecretan this could be blocking inventory from being served to users. How about we automatically remove blocks after 90 days and users can just block again?

@jsecretan
Copy link

I don't think I'm ready to do that just yet. And really inventory should cycle mostly in that time. I'm willing to hold on this issue right now

@tmancey
Copy link
Contributor

tmancey commented Jun 14, 2022

@jsecretan sure but unfortunately inventory cycling will not help as users are able to block categories and once blocked they can never unblock. Users can also block untargeted which is the reason this issue was raised.

@tmancey tmancey added the needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. label Jun 24, 2022
@tmancey tmancey assigned tmancey and unassigned minkyeongshin Jun 14, 2023
@tmancey tmancey added closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. and removed blocked labels Jun 14, 2023
@tmancey
Copy link
Contributor

tmancey commented Jun 14, 2023

Closing issue as stale, however making @Miyayes aware for future consideration in new ongoing design changes

@tmancey tmancey closed this as completed Jun 14, 2023
@tmancey tmancey added this to Ads Jun 10, 2024
@tmancey tmancey moved this to Done in Ads Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. design A design change, especially one which needs input from the design team enhancement feature/ads feature/rewards needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. OS/Desktop
Projects
Archived in project
Development

No branches or pull requests

6 participants