-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Country not part of ad confirmation call - follow up to #8100 #9823
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/Yes
release-notes/include
Milestone
Comments
tmancey
added
priority/P3
The next thing for us to work on. It'll ride the trains.
bug
labels
May 25, 2020
32 tasks
Marked it as |
This was referenced Jul 8, 2020
Removing |
Verification in progress (pending question)
Screenshots where countryCode is includedcountryCode = US (United States)
Verification passed on
Verification passed on
FR
AU
|
rebron
changed the title
country not part of ad confirmation call - follow up to 8100
Country not part of ad confirmation call - follow up to #8100
Jul 15, 2020
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/Yes
release-notes/include
Description
Found while testing #8100
Per above issue,
countryCode
should be present (if country is large enough) on release builds for ad confirmations.Steps to Reproduce
--enable-logging=stderr --vmodule=brave_ads=3 --log-level=2
) and be running CharlesActual result:
build channel and platform are listed, but not countryCode:
Expected result:
Since 1.9.70 is on Release channel and my region = US in my OS settings, I should have
countryCode
in the above, but it is not there.Reproduces how often:
easily
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc @tmancey @jsecretan @rebron @brave/legacy_qa
The text was updated successfully, but these errors were encountered: