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

Uphold funds AC- AC is not recorded in summary and monthly statements #10038

Closed
GeetaSarvadnya opened this issue Jun 1, 2020 · 2 comments
Closed
Labels
closed/duplicate Issue has already been reported feature/rewards

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 1, 2020

Description

Found while testing #7595 (scenario 1 from brave/brave-core#4869)
AC is not recorded in summary and monthly statements

Steps to Reproduce

  1. start the browser with a short interval
  2. enable rewards
  3. connect to KYC uphold
  4. add some verified sites to AC table
  5. wait for AC to be triggered
  6. No AC notification on BAT logo
  7. AC is not recorded in summary and monthly statements
  8. Logs are not recorded for /v1/autocontribute/votes
  9. AC amount was taken from KYC'd uphold wallet

Note: KYC'd uphold wallet is self-funded via sandbox uphold account

Actual result:

image

image

Expected result:

AC should be recorded in summary and monthly statements

Reproduces how often:

Always

Brave version (brave://version info)

Brave 1.10.79 Chromium: 81.0.4044.138 (Official Build) dev (64-bit)
Revision 8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{#999}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Version/Channel Information:

  • Can you reproduce this issue with the current release? NA
  • Can you reproduce this issue with the beta channel? not sure
  • Can you reproduce this issue with the dev channel? Yes
  • Can you reproduce this issue with the nightly channel? Not sure

Other Additional Information:

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

Miscellaneous Information:

cc: @NejcZdovc @brave/legacy_qa @rebron

@GeetaSarvadnya GeetaSarvadnya added this to the 1.10.x - Beta milestone Jun 1, 2020
@GeetaSarvadnya GeetaSarvadnya changed the title AC is not recorded in summary and monthly statements Uphold funds AC- AC is not recorded in summary and monthly statements Jun 1, 2020
@NejcZdovc
Copy link
Contributor

can you please re-test it when this one lands in 1.10 #9962. It was merged 4h ago in 1.10

@kjozwiak
Copy link
Member

kjozwiak commented Jun 2, 2020

Closing as per the above 👍 @GeetaSarvadnya please recheck once brave/brave-core#5692 lands in 1.10.x as brave/brave-core#5694 has been uplifted.

@kjozwiak kjozwiak closed this as completed Jun 2, 2020
@NejcZdovc NejcZdovc removed this from the 1.10.x - Beta milestone Jun 2, 2020
@NejcZdovc NejcZdovc added closed/duplicate Issue has already been reported and removed QA/Test-Plan-Specified QA/Yes bug labels Jun 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/duplicate Issue has already been reported feature/rewards
Projects
None yet
Development

No branches or pull requests

3 participants