-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Full country list displayed for Rewards when only Japan is expected in certain case - Follow up to 28887 and 28321 #29609
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
Labels
feature/rewards
OS/Desktop
priority/P4
Planned work. We expect to get to it "soon".
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-All-Platforms
QA/Yes
release-notes/exclude
Milestone
Comments
cc @GeetaSarvadnya @Uni-verse to check on Android since original issue had |
Different issue occurs on Android, @GeetaSarvadnya to log an issue for it. |
25 tasks
Merged
7 tasks
The above requires |
Verification PASSED on
Case 1 - Clean install, non-Japan region - PASSED
Case 2A - Clean install, Japan region - PASSED
Case 2B - Reset Rewards, Japan region - PASSED
Case 3 - Upgrade (connected) from version without geo-declaration, non-Japan region - PASSED
Case 4 - Upgrade (connected) from version without geo-declaration, Japan region - PASSED
Case 5 - Upgrade (grandfathered unverified) from version without geo-declaration, Japan region - PASSED
|
Verified with
Case 1 - Clean install, non-Japan region - PASSED
Case 2A - Clean install, Japan region - PASSED
Case 2B - Reset Rewards, Japan region - PASSED
Case 3 - Upgrade (connected) from version without geo-declaration, non-Japan region - PASSED
Case 4 - Upgrade (connected) from version without geo-declaration, Japan region - PASSED
Case 5 - Upgrade (grandfathered unverified) from version without geo-declaration, Japan region - PASSED
|
Verification passed on
Case 1 - Clean install, non-Japan region - PASSED
Case 2A - Clean install, Japan region - PASSED
Case 2B - Reset Rewards, Japan region - PASSED
Case 3 - Upgrade (connected) from version without geo-declaration, non-Japan region - PASSED
Case 4 - Upgrade (connected) from version without geo-declaration, Japan region - PASSED
Case 5 - Upgrade (grandfathered unverified) from version without geo-declaration, Japan region - PASSED
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature/rewards
OS/Desktop
priority/P4
Planned work. We expect to get to it "soon".
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-All-Platforms
QA/Yes
release-notes/exclude
Description
Found while testing #28887 and #28321
Test case 2 from brave/brave-core#17468 and test case 2 from brave/brave-core#17427 for above issues respectivly failed. Test cases are outlined below.
Note, references to bold are not applicable to macOS per discussion with @Miyayes.
Steps to Reproduce
Scenario A: Test Case 2 from brave/brave-core#17468:
Per test plan, start with old version where we don't have geo declaration yet (I used 1.45.104):
1. connect your bitFlyer account
2. upgrade
3. set your OS region to Japan
4. restart browser
5. make sure Japan is the only item in the list (should be bold if not on macOS)
6. set your OS region to the US
7. restart browser
---> Japan should be the only item in the list at this point (not bold no matter which OS you're using) per brave/brave-core#17468 (comment), however full list of countries is displayed.
Scenario B: Test Case 2 from brave/brave-core#17427:
Per test plan, start with old version where we don't have geo declaration yet (I used 1.45.104):
1. set your OS region to the US
2. install Brave and launch using staging parameters and the
countryid
flag (meaning, make sure your--rewards
args includecountryid=19024
for Japan. Example:--rewards=staging=true,debug=true,countryid=19024
)3. connect your bitFlyer account
4. upgrade to 1.51.x
---> make sure US is not listed in the country dropdown list --> this fails.
Reproduces how often:
easily
Brave version (brave://version info)
1.51.80 Chromium: 112.0.5615.49 (Official Build) beta (x86_64)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc @Miyayes
The text was updated successfully, but these errors were encountered: