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

Additional phase of Brave Ads regions #7817

Closed
tmancey opened this issue Jan 19, 2020 · 6 comments · Fixed by brave/brave-core#4427
Closed

Additional phase of Brave Ads regions #7817

tmancey opened this issue Jan 19, 2020 · 6 comments · Fixed by brave/brave-core#4427

Comments

@tmancey
Copy link
Contributor

tmancey commented Jan 19, 2020

Add the following regions to Brave Ads:

Afghanistan | AF
American Samoa | AS
Andorra | AD
Angola | AO
Anguilla | AI
Antarctica | AQ
Antigua and Barbuda | AG
Armenia | AM
Aruba | AW
Bahamas | BS
Bahrain | BH
Barbados | BB
Belarus | BY
Belize | BZ
Benin | BJ
Bermuda | BM
Bhutan | BT
Bolivia | BO
Bonaire | BQ
Bosnia and Herzegovina | BA
Botswana | BW
Bouvet Island | BV
British Indian Ocean Territory | IO
Brunei Darussalam | BN
Burkina Faso | BF
Burundi | BI
Cameroon | CM
Cape Verde | CV
Central African Republic | CF
Chad | TD
Christmas Island | CX
Cocos (Keeling) Islands | CC
Comoros | KM
Congo | CG
Democratic Republic of the Congo | CD
Cook Islands | CK
Costa Rica | CR
Curacao | CW
Cyprus | CY
Cote d'Ivoire | CI
Djibouti | DJ
Dominica | DM
Dominican Republic | DO
El Salvador | SV
Equatorial Guinea | GQ
Eritrea | ER
Estonia | EE
Ethiopia | ET
Falkland Islands (Malvinas) | FK
Faroe Islands | FO
Fiji | FJ
French Guiana | GF
French Polynesia | PF
French Southern Territories | TF
Gabon | GA
Gambia | GM
Georgia | GE
Ghana | GH
Gibraltar | GI
Greenland | GL
Grenada | GD
Guadeloupe | GP
Guam | GU
Guatemala | GT
Guernsey | GG
Guinea | GN
Guinea-Bissau | GW
Guyana | GY
Haiti | HT
Heard Island and McDonald Islands | HM
Holy See (Vatican City State) | VA
Honduras | HN
Iceland | IS
Isle of Man | IM
Jamaica | JM
Jersey | JE
Jordan | JO
Kazakhstan | KZ
Kenya | KE
Kiribati | KI
Kuwait | KW
Kyrgyzstan | KG
Lao People's Democratic Republic | LA
Latvia | LV
Lebanon | LB
Lesotho | LS
Liberia | LR
Libya | LY
Liechtenstein | LI
Luxembourg | LU
Macao | MO
Macedonia, the Former Yugoslav Republic of | MK
Madagascar | MG
Malawi | MW
Maldives | MV
Mali | ML
Malta | MT
Marshall Islands | MH
Martinique | MQ
Mauritania | MR
Mauritius | MU
Mayotte | YT
Micronesia, Federated States of | FM
Moldova, Republic of | MD
Monaco | MC
Mongolia | MN
Montenegro | ME
Montserrat | MS
Mozambique | MZ
Myanmar | MM
Namibia | NA
Nauru | NR
Nepal | NP
New Caledonia | NC
Nicaragua | NI
Niger | NE
Niue | NU
Norfolk Island | NF
Northern Mariana Islands | MP
Norway | NO
Oman | OM
Palau | PW
Palestine, State of | PS
Panama | PA
Papua New Guinea | PG
Paraguay | PY
Pitcairn | PN
Puerto Rico | PR
Qatar | QA
Rwanda | RW
Reunion | RE
Saint Barthelemy | BL
Saint Helena | SH
Saint Kitts and Nevis | KN
Saint Lucia | LC
Saint Martin (French part) | MF
Saint Pierre and Miquelon | PM
Saint Vincent and the Grenadines | VC
Samoa | WS
San Marino | SM
Sao Tome and Principe | ST
Senegal | SN
Seychelles | SC
Sierra Leone | SL
Sint Maarten (Dutch part) | SX
Solomon Islands | SB
Somalia | SO
South Georgia and the South Sandwich Islands | GS
South Sudan | SS
Sri Lanka | LK
Sudan | SD
Suriname | SR
Svalbard and Jan Mayen | SJ
Swaziland | SZ
Tajikistan | TJ
United Republic of Tanzania | TZ
Timor-Leste | TL
Togo | TG
Tokelau | TK
Tonga | TO
Trinidad and Tobago | TT
Tunisia | TN
Turkmenistan | TM
Turks and Caicos Islands | TC
Tuvalu | TV
Uganda | UG
United States Minor Outlying Islands | UM
Uruguay | UY
Uzbekistan | UZ
Vanuatu | VU
British Virgin Islands | VG
US Virgin Islands | VI
Wallis and Futuna | WF
Western Sahara | EH
Yemen | YE
Zambia | ZM
Zimbabwe | ZW

@tmancey tmancey added priority/P2 A bad problem. We might uplift this to the next planned release. feature/ads labels Jan 19, 2020
@tmancey tmancey self-assigned this Jan 19, 2020
@tmancey tmancey added priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes enhancement and removed priority/P2 A bad problem. We might uplift this to the next planned release. labels Jan 19, 2020
@jwchong
Copy link

jwchong commented Mar 10, 2020

Pls add Malaysia into the region list.

@eljuno
Copy link

eljuno commented Mar 10, 2020

@jwchong Malaysia is already supported (#6825) -- https://brave.com/transparency/

@jsecretan
Copy link

jsecretan commented Mar 24, 2020

For privacy reasons, we should put the follow country codes together under a country code for Other "??":

AS
AI
AQ
AG
BQ
BV
IO
CF
TD
CX
CC
KM
CK
GQ
ER
FK
TF
GL
GW
HM
VA
KI
LR
MH
YT
FM
MS
NR
NU
NF
MP
PW
PN
BL
SH
KN
MF
PM
WS
SM
ST
SX
SB
GS
SS
SJ
TJ
TL
TK
TM
TC
TV
UM
VU
VG
WF
EH

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented May 5, 2020

@jsecretan @tmancey Cannot verify the issue as catalog is empty for additional ad regions, for more details please refer https://bravesoftware.slack.com/archives/CJL3G0S95/p1588687877157000

@LaurenWags
Copy link
Member

LaurenWags commented May 18, 2020

Test Plan:

@LaurenWags
Copy link
Member

LaurenWags commented May 19, 2020

Verified passed using

Brave	1.9.70 Chromium: 81.0.4044.138 (Official Build) (64-bit)
Revision	8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{#999}
OS	macOS Version 10.14.6 (Build 18G3020)
  • Verified the following for new regions (Estonia, Moldova)
    • Clean install of 1.9.x. Enable Rewards. Confirmed Ads are enabled by default. Confirmed able to view an ad.
    • Installed 1.8.96. Enabled Rewards. Confirmed Ads are not available. Upgraded to 1.9.70. Confirmed notification displayed on BAT logo to alert user that ads are now available in their region. Confirmed Ads are not enabled by default on upgrade. Confirmed able to enable ads and view an ad.

Estonia on 1.8.96:
Estonia 1 8 x

Estonia after upgrade to 1.9.x:
estonia upgrade 1
estonia upgrade 2

Confirming ad panel updates for Estonia when ads are enabled and an ad notification has been served:
estonia ad panel updates

Moldova on 1.8.96:
Moldova 1 8 x

Moldova after upgrade to 1.9.x:
Moldova upgrade 1
Moldova upgrade 2

Confirming ad panel updates for Moldova when ads are enabled and an ad notification has been served:
Moldova ad panel updates

  • Also per Additional phase of Brave Ads regions #7817 (comment), spot checked previously enabled ad regions: US, Ireland, Russia and confirmed the following scenarios:
    • Clean install of 1.9.x. Enable Rewards. Confirmed Ads are enabled by default.
    • Installed 1.8.96. Enabled Rewards. Disabled Ads. Upgraded to 1.9.x. Confirmed Ads are still off after upgrade. No notification that ads are available.
    • Installed 1.8.96. Enabled Rewards. Upgraded to 1.9.x. Confirmed Ads are still on after upgrade. No notification that ads are available.

Verification passed on

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

  • Verified the following for new regions Iceland and Latvia

    • Clean install of 1.9.x. Enable Rewards. Confirmed Ads are enabled by default. Confirmed able to view an ad.
      image

    • Installed 1.8.96. Enabled Rewards. Confirmed Ads are not available. Upgraded to 1.9.70. Confirmed notification displayed on BAT logo to alert user that ads are now available in their region. Confirmed Ads are not enabled by default on upgrade. Confirmed able to enable ads and view an ad.

Iceland 1.8.96:
image

Icelanda after upgrade to 1.9.71
image

Confirming ad panel updates for Iceland when ads are enabled and an ad notification has been served:
image

  • Clean install of 1.9.x. Enable Rewards. Confirmed Ads are enabled by default. Confirmed able to view an ad.
    image

  • Installed 1.8.96. Enabled Rewards. Confirmed Ads are not available. Upgraded to 1.9.70. Confirmed notification displayed on BAT logo to alert user that ads are now available in their region. Confirmed Ads are not enabled by default on upgrade. Confirmed able to enable ads and view an ad.

Latvia 1.8.96:
image

Latvia after upgrade to 1.9.71:
image

Confirming ad panel updates for Latvia when ads are enabled and an ad notification has been served:
image

  • Also per Additional phase of Brave Ads regions #7817 (comment), spot checked previously enabled ad regions: US and India and confirmed the following scenarios:
    • Clean install of 1.9.x. Enable Rewards. Confirmed Ads are enabled by default.
    • Installed 1.8.96. Enabled Rewards. Disabled Ads. Upgraded to 1.9.x. Confirmed Ads are still off after upgrade. No notification that ads are available.
    • Installed 1.8.96. Enabled Rewards. Upgraded to 1.9.x. Confirmed Ads are still on after upgrade. No notification that ads are available.

Verification passed on

Brave 1.9.71 Chromium: 81.0.4044.138 (Official Build) (64-bit)
Revision 8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{#999}
OS Ubuntu 18.04 LTS

Verified ads are being enabled after upgrade
Verified ads are enabled on clean install
Verified ability to view an ad
Verified the following regions:

Liechtenstein | LI in 1.8.x
image
Liechtenstein | LI in 1.9.x
image

Norway | NO in 1.8.x
image
Norway | NO in 1.9.x
image

Luxembourg | LU in 1.8.x
image
Luxembourg | LU in 1.9.x
image

Also per #7817 (comment), spot checked previously enabled ad regions: US, Poland. Confirmed the following scenarios:
Clean install of 1.9.x. Enable Rewards. Confirmed Ads are enabled by default.
Installed 1.8.96. Enabled Rewards. Disabled Ads. Upgraded to 1.9.x. Confirmed Ads are still off after upgrade. No notification that ads are available.
Installed 1.8.96. Enabled Rewards. Upgraded to 1.9.x. Confirmed Ads are still on after upgrade. No notification that ads are available.

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

Successfully merging a pull request may close this issue.

7 participants