Skip to content

Positive Attention value in Rewards Panel despite minimum page time requirement not being met #3995

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

Closed
btlechowski opened this issue Apr 4, 2019 · 5 comments

Comments

@btlechowski
Copy link

Steps to Reproduce

  1. Clean profile
  2. Enable rewards
  3. Go to interia.pl and wait till the page is added to a-c table
  4. Change Minimum page time before logging a visit to 1 minute
  5. Go to Interia.pl
  6. Open Rewards Panel

Actual result:

Positive Attention value

image

Expected result:

0% Attention

image

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 0.63.30 Chromium: 73.0.3683.75 (Official Build) beta (64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Windows 10 OS Build 17134.523
Brave 0.65.16 Chromium: 74.0.3729.40 (Official Build) nightly (64-bit)
Revision 278d6a0d856d8314f36fb9a68b6e1d85cd0d14e7-refs/branch-heads/3729@{#470}
OS Windows 10 OS Build 17134.523

cc @brave/legacy_qa @NejcZdovc

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Apr 4, 2019

@btlechowski do you see any entry in ac table as well or the only problem is in the panel%?

@btlechowski
Copy link
Author

@NejcZdovc interia.pl is not in a-c table as expected. The a-c table is empty as expected. It is only problem with Rewards Panel.

@NejcZdovc
Copy link
Contributor

ok thank you

@NejcZdovc NejcZdovc added the priority/P3 The next thing for us to work on. It'll ride the trains. label Apr 5, 2019
jasonrsadler pushed a commit to brave/brave-core that referenced this issue Apr 10, 2019
@jasonrsadler
Copy link

jasonrsadler commented May 29, 2019

Update. When setting min time to 1 min, publisher stays in ac list until refresh:
rewards_service_impl.cc(2885)] Problem saving normalized publishers in SaveNormalizedPublisherList

Fixed with brave/brave-core#2520

@jasonrsadler jasonrsadler removed their assignment Jul 29, 2019
@NejcZdovc
Copy link
Contributor

closing as fixed with brave/brave-core#2520

@NejcZdovc NejcZdovc added this to the Dupe / Invalid / Not actionable milestone Apr 10, 2020
@NejcZdovc NejcZdovc removed QA/Test-Plan-Specified QA/Yes priority/P3 The next thing for us to work on. It'll ride the trains. labels Apr 10, 2020
@NejcZdovc NejcZdovc modified the milestones: Dupe / Invalid / Not actionable, 0.68.x - Release Apr 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants