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

Brave Ads history popup data appears to clear out in certain cases, reopen fixes it #6206

Closed
LaurenWags opened this issue Sep 27, 2019 · 4 comments
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated.

Comments

@LaurenWags
Copy link
Member

Description

If you are viewing 'Saved' ads on the Ads History popup and you remove all saved ads, the Ads History popup data appears to be cleared out. However, if you close and reopen the popup, all data is there as expected.

Steps to Reproduce

  1. View/click/dismiss some ads.
  2. View Ad History popup.
  3. Mark at least one ad as saved.
  4. Click on Saved.
  5. The popup switches to showing only the Saved ad in the view.
  6. Remove the ad from Saved (click on hamburger menu > remove)

Actual result:

All data is cleared:
adspanel-blank

Expected result:

I was expecting to be directed back to the main view and see all my ad history.

Reproduces how often:

easily

Brave version (brave://version info)

Brave 0.69.129 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the dev channel? yes
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Does the issue resolve itself when disabling Brave Rewards? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Miscellaneous Information:

cc @alexwykoff

@btlechowski
Copy link

Reproduced on Ubuntu.

Brave 0.69.131 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Ubuntu 18.04 LTS

image

@jsecretan jsecretan added the priority/P4 Planned work. We expect to get to it "soon". label Nov 21, 2019
@tmancey tmancey removed the priority/P4 Planned work. We expect to get to it "soon". label Jan 20, 2020
@tmancey tmancey changed the title Ad History popup data appears to clear out in certain cases, reopen fixes it Brave Ads history popup data appears to clear out in certain cases, reopen fixes it Jan 20, 2020
@tmancey tmancey added bug and removed QA/Yes labels Jan 20, 2020
@tmancey tmancey added priority/P3 The next thing for us to work on. It'll ride the trains. release-notes/include QA/Yes good first issue labels May 29, 2020
@yachtcaptain23 yachtcaptain23 self-assigned this Jun 10, 2020
@tmancey tmancey added OS/Desktop and removed QA/Yes priority/P3 The next thing for us to work on. It'll ride the trains. release-notes/include labels Aug 5, 2020
@tmancey tmancey added the OS/Android Fixes related to Android browser functionality label Aug 20, 2020
@tmancey tmancey removed the OS/Android Fixes related to Android browser functionality label Mar 7, 2021
@jeffbum
Copy link

jeffbum commented Jan 15, 2022

@LaurenWags if this is an issue, I would like to work on this.

@LaurenWags
Copy link
Member Author

Hi @jeffbum! Let me get confirmation this issue still reproduces before you start on it.

@LaurenWags
Copy link
Member Author

Looks like this one no longer reproduces using the below version. Thanks for your interest @jeffbum. Please feel free to review the other open issues and see if one looks interesting to you.

Brave | 1.36.44 Chromium: 97.0.4692.71 (Official Build) nightly (x86_64)
-- | --
Revision | adefa7837d02a07a604c1e6eff0b3a09422ab88d-refs/branch-heads/4692@{#1247}
OS | macOS Version 11.6.1 (Build 20G224)

@LaurenWags LaurenWags added closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. and removed bug good first issue feature/ads OS/Desktop labels Jan 18, 2022
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.
Projects
None yet
Development

No branches or pull requests

7 participants