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

Insufficient funds notification when there is no site in a-c list #4104

Closed
srirambv opened this issue Apr 15, 2019 · 3 comments · Fixed by brave/brave-core#2208
Closed

Insufficient funds notification when there is no site in a-c list #4104

srirambv opened this issue Apr 15, 2019 · 3 comments · Fixed by brave/brave-core#2208

Comments

@srirambv
Copy link
Contributor

Description

Insufficient funds notification when there is no site in a-c list

Steps to Reproduce

  1. Enable rewrads
  2. Do not claim any grants
  3. Do not add any sites to a-c list
  4. Trigger reconcile
  5. Insufficient funds notification shown

Actual result:

Insufficient funds notification when there is no site in a-c list

Expected result:

Should not show insufficient funds message when there is no site added to a-c list

Reproduces how often:

Easy

Additional Information

cc: @NejcZdovc

@NejcZdovc NejcZdovc assigned NejcZdovc and jasonrsadler and unassigned NejcZdovc Apr 18, 2019
@NejcZdovc NejcZdovc added the priority/P3 The next thing for us to work on. It'll ride the trains. label Apr 18, 2019
@jasonrsadler
Copy link

@srirambv Can you see if this is still an issue (at least on nightly). I do get one 'Insufficient' notification in the panel but that is before reconciliation starts and is fixed in brave/brave-core#2208 (pending). Can you check by:

  • running with term with reconcile-interval in clean profile,
  • dismissing grant,
  • waiting a minute or two and dismissing 'Insufficient' (before reconcile triggers[you may have to click off and back on the panel to get notifications])
  • and then viewing terminal and waiting for Auto contribution table is empty and then see if notification still appears at that point?

jasonrsadler pushed a commit to brave/brave-core that referenced this issue May 14, 2019
jasonrsadler pushed a commit to brave/brave-core that referenced this issue May 14, 2019
@srirambv
Copy link
Contributor Author

@jasonrsadler Tried the following on 0.67.31 Chromium: 75.0.3770.27 (Official Build) nightly (64-bit)

  • Launch browser with brave-browser-nightly --enable-logging=stderr --rewards=staging=true,reconcile_interval=2 --vmodule=*rewards*=2
  • Enable rewards and not claim grant
  • Wait for 2 mins for reconcile_interval to do its magic
  • No notification shown for insufficient funds
  • Didn't see any console message for Auto contribution table is empty as well

jasonrsadler pushed a commit to brave/brave-core that referenced this issue May 16, 2019
jasonrsadler pushed a commit to brave/brave-core that referenced this issue May 20, 2019
jasonrsadler pushed a commit to brave/brave-core that referenced this issue May 21, 2019
@LaurenWags
Copy link
Member

LaurenWags commented Jul 12, 2019

Verified passed with

Brave 0.67.108 Chromium: 75.0.3770.100 (Official Build) beta(64-bit)
Revision cd0b15c8b6a4e70c44e27f35c37a4029bad3e3b0-refs/branch-heads/3770@{#1033}
OS Mac OS X

Verification passed on

Brave 0.67.108 Chromium: 75.0.3770.100 (Official Build) beta (64-bit)
Revision cd0b15c8b6a4e70c44e27f35c37a4029bad3e3b0-refs/branch-heads/3770@{#1033}
OS Windows 10 OS Version 1803 (Build 17134.523)

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