-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
BAT confirmations are initialized on clean profile #10085
Labels
bug
feature/ads
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-Plan-Specified
QA/Yes
release-notes/include
Milestone
Comments
tmancey
added
bug
priority/P3
The next thing for us to work on. It'll ride the trains.
labels
Jun 4, 2020
Fixed in 1.11.x |
@tmancey can we close this issue? |
Also reproduced on upgrade
Failed on upgrade from |
Verified passed with
Verification passed on
Clean profile:
Upgraded profile:
Verified passed with
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
feature/ads
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-Plan-Specified
QA/Yes
release-notes/include
Follow up to #9544
Steps to Reproduce
In
1.10.x
use--enable-logging=stderr --vmodule=brave_ads=3
in
1.11.x
use--enable-logging=stderr --vmodule="*/bat-native-ads/*"=6,"*/bat-native-confirmations/*"=6,"*/bat-native-ledger/*"=6
Initializing confirmations
Default/rewards_service/confirmations.json
is createdActual result:
Initializing confirmations
is recorded in the logsDefault/rewards_service/confirmations.json
is createdExpected result:
Initializing confirmations
is not recorded in the logsDefault/rewards_service/confirmations.json
is not createdReproduces how often:
100% repro rate
Brave version (brave://version info)
cc @brave/legacy_qa @tmancey
The text was updated successfully, but these errors were encountered: