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

Remove "Ready to start earning?" modal and update custodial selection modal #29448

Closed
Miyayes opened this issue Mar 31, 2023 · 2 comments · Fixed by brave/brave-core#17949
Closed
Assignees
Labels
feature/rewards OS/Android Fixes related to Android browser functionality OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass - Android ARM QA Pass-macOS QA/Yes release-notes/include

Comments

@Miyayes
Copy link
Collaborator

Miyayes commented Mar 31, 2023

Description

Consolidate these two modals into a single step (keep the custodial selection modal).

On the custodial selection modal:

(1) Remove "If you already have..." paragraph

(2) Add these bullet points:
image

But change "Tip your favorite..." to "Send on-demand contributions to..."

(3) Modify disclaimer at bottom to say:

Custodial account providers are required to verify your identity, which may include submitting a photo ID. See the Brave Rewards Privacy Policy. See here to learn more about BAT.

@Miyayes Miyayes added OS/Android Fixes related to Android browser functionality OS/Desktop feature/rewards labels Mar 31, 2023
@Miyayes Miyayes added the priority/P3 The next thing for us to work on. It'll ride the trains. label Mar 31, 2023
@emerick emerick added this to the 1.52.x - Nightly milestone Apr 13, 2023
@LaurenWags LaurenWags added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label May 4, 2023
@LaurenWags
Copy link
Member

LaurenWags commented May 4, 2023

Verified with

Brave | 1.52.86 Chromium: 113.0.5672.77 (Official Build) beta (x86_64)
-- | --
Revision | c4236862955e005c2187105415ac4a2ecf86dff1-refs/branch-heads/5672_62@{#3}
OS | macOS Version 13.3.1 (Build 22E261)

Verified the test plan from the PR, brave/brave-core#17949 (comment), for the below scenarios on this modal.

2 custodians available - PASSED

Confirmed that country selected prior to joining (staging) Rewards (I used United States) is in the allow list for both Uphold and Gemini on staging:

[ RESPONSE - OnResponse ]
> Url: https://api.rewards.bravesoftware.com/v1/parameters
> Result: Success
> HTTP Code: 200
> Body: {"autocontribute":{"choices":[1,2,3,5,7,10,20],"defaultChoice":1},"batRate":0.220262,"custodianRegions":{"bitflyer":{"allow":["JP"],"block":[]},"gemini":{"allow":["AU","AT","BE","CA","CO","DK","FI","HK","IE","IT","NL","NO","PL","PT","SG","ES","SE","GB","US"],"block":[]},"uphold":{"allow":["AU","AT","BE","CA","CO","DK","FI","HK","IE","IT","NL","NO","PL","PT","SG","ES","SE","GB","US","IN"],"block":[]}},"payoutStatus":{"bitflyer":"complete","gemini":"complete","payoutDate":"","unverified":"complete","uphold":"complete"},"tips":{"defaultMonthlyChoices":[1.25,5,10.5],"defaultTipChoices":[1.25,5,10.5]},"vbatDeadline":"2023-02-10T00:00:00Z","vbatExpired":false}

Confirmed modal looked like the one in the PR:

2
Uphold custodian available, Gemini unavailable - PASSED

Confirmed that country selected prior to joining (staging) Rewards (I used India) is in the allow list for Uphold but not Gemini on staging:

[ RESPONSE - OnResponse ]
> Url: https://api.rewards.bravesoftware.com/v1/parameters
> Result: Success
> HTTP Code: 200
> Body: {"autocontribute":{"choices":[1,2,3,5,7,10,20],"defaultChoice":1},"batRate":0.220262,"custodianRegions":{"bitflyer":{"allow":["JP"],"block":[]},"gemini":{"allow":["AU","AT","BE","CA","CO","DK","FI","HK","IE","IT","NL","NO","PL","PT","SG","ES","SE","GB","US"],"block":[]},"uphold":{"allow":["AU","AT","BE","CA","CO","DK","FI","HK","IE","IT","NL","NO","PL","PT","SG","ES","SE","GB","US","IN"],"block":[]}},"payoutStatus":{"bitflyer":"complete","gemini":"complete","payoutDate":"","unverified":"complete","uphold":"complete"},"tips":{"defaultMonthlyChoices":[1.25,5,10.5],"defaultTipChoices":[1.25,5,10.5]},"vbatDeadline":"2023-02-10T00:00:00Z","vbatExpired":false}

Confirmed modal looked like the one in the PR:

India
Gemini custodian available, Uphold unavailable - PASSED

Used Charles Proxy to rewrite the response body for the parameters endpoint so that United Kingdom (GB) was not in the "allow" list for Uphold:

[ RESPONSE - OnResponse ]
> Url: https://api.rewards.bravesoftware.com/v1/parameters
> Result: Success
> HTTP Code: 200
> Body: {"autocontribute":{"choices":[1,2,3,5,7,10,20],"defaultChoice":1},"batRate":0.211087,"custodianRegions":{"bitflyer":{"allow":["JP"],"block":[]},"gemini":{"allow":["AU","AT","BE","CA","CO","DK","FI","HK","IE","IT","NL","NO","PL","PT","SG","ES","SE","GB","US"],"block":[]},"uphold":{"allow":["AU","AT","BE","CA","CO","DK","FI","HK","IE","IT","NL","NO","PL","PT","SG","ES","SE","US","IN"],"block":[]}},"payoutStatus":{"bitflyer":"complete","gemini":"complete","payoutDate":"","unverified":"complete","uphold":"complete"},"tips":{"defaultMonthlyChoices":[1.25,5,10.5],"defaultTipChoices":[1.25,5,10.5]},"vbatDeadline":"2023-02-10T00:00:00Z","vbatExpired":false}

Confirmed modal looked like the one in the PR:

GB
Only 1 custodian (bitFlyer) available - PASSED

Confirmed that country selected prior to joining (staging) Rewards (Japan) is in the allow list for bitFlyer on staging:

[ RESPONSE - OnResponse ]
> Url: https://api.rewards.bravesoftware.com/v1/parameters
> Result: Success
> HTTP Code: 200
> Body: {"autocontribute":{"choices":[1,2,3,5,7,10,20],"defaultChoice":1},"batRate":0.210463,"custodianRegions":{"bitflyer":{"allow":["JP"],"block":[]},"gemini":{"allow":["AU","AT","BE","CA","CO","DK","FI","HK","IE","IT","NL","NO","PL","PT","SG","ES","SE","GB","US"],"block":[]},"uphold":{"allow":["AU","AT","BE","CA","CO","DK","FI","HK","IE","IT","NL","NO","PL","PT","SG","ES","SE","GB","US","IN"],"block":[]}},"payoutStatus":{"bitflyer":"complete","gemini":"complete","payoutDate":"","unverified":"complete","uphold":"complete"},"tips":{"defaultMonthlyChoices":[1.25,5,10.5],"defaultTipChoices":[1.25,5,10.5]},"vbatDeadline":"2023-02-10T00:00:00Z","vbatExpired":false}

Confirmed modal looked like the one in the PR:

bF

@LaurenWags LaurenWags added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels May 8, 2023
@hffvld hffvld added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label May 23, 2023
@hffvld
Copy link
Contributor

hffvld commented May 24, 2023

Verified on Google Pixel 7 using version(s):

Device/OS: Google Pixel 7 [panther_beta-user 14 UPB2.230407.014 release-keys]
Brave build: 1.52.110
Chromium: 114.0.5735.35 (Official Build) (64-bit)
Revision: 7429f911c043513bd013a56876f67313d91dcbfd-refs/branch-heads/5672_132@{#8}

STEPS:

  1. Clean profile
  2. Enable Rewards > Select country
  3. Tap Connect account button in Rewards panel

ACTUAL RESULTS:

  • Verified that Connect modal looks/behaves as described in issue
2 custodians available (US) - PASSED
1 2
1 2
Uphold custodian available, Gemini unavailable (India) - PASSED
1 2
1 2
Only 1 custodian (bitFlyer) available (Japan) - PASSED
1 2
1 2

@hffvld hffvld added QA Pass - Android ARM and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/rewards OS/Android Fixes related to Android browser functionality OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass - Android ARM QA Pass-macOS QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants