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

Reduce Uphold minimum BAT user wallet verification requirement to 15 BAT #15055

Closed
Miyayes opened this issue Mar 31, 2021 · 6 comments · Fixed by brave/brave-core#8453
Closed

Comments

@Miyayes
Copy link
Collaborator

Miyayes commented Mar 31, 2021

Description

Currently, users need to have 25 BAT to verify their user wallet with Uphold.

Uphold has approved lowering the verification threshold to 15 BAT as of March 31, 2021.

Solution

Reduce all relevant wallet verification gating logic to a 15 BAT threshold.

@CloggyAbroad
Copy link

It looks like the text hasn't been updated in all locations

image

@stephendonner
Copy link

stephendonner commented Apr 21, 2021

Verified PASSED using the testplan from brave/brave-core#8453

Brave 1.25.33 Chromium: 90.0.4430.85 (Official Build) nightly (x86_64)
Revision 5bc145d831c180d9ff94f29a0d7a2e1cbd30ef36-refs/branch-heads/4430@{#1311}
OS macOS Version 11.2.3 (Build 20D91)

Steps:

Scenario: User has less than 15 BAT (Rewards panel & brave://rewards page)

  1. launched Brave using --enable-logging=stderr --vmodule="*/bat-native-ledger/*"=6,"*/brave_rewards/*"=6,"*/bat-native-ads/*"=6,"*/bat-native-confirmations/*"=6,"*/brave_ads/*"=9,"*/brave_user_model/*"=6 --brave-ads-staging --brave-ads-debug --rewards=staging=true,reconcile-interval=4
  2. clicked Start using Rewards and confirmed UGP token grant of 30.000 BAT
  3. loaded https://www.github.com/laurenwags, and clicked on the BAT icon
  4. clicked Send a tip...
  5. chose 20 BAT and clicked Send Tip
  6. BAT balance was now 10.000 BAT
  7. clicked on the BAT icon again and clicked Verify Wallet
  8. confirmed that the text about minimum BAT to create an Uphold account read 15 BAT
  9. loaded brave://rewards and clicked on the Verify Wallet link
  10. confirmed also that the text there about minimum BAT to create an Uphold account read 15 BAT
  11. from both entry-points, I clicked on the orange Login button and was taken to Uphold's sandbox (https://wallet-sandbox.uphold.com/)
  12. verified I could log in to Uphold, but my wallet wasn't verified

Scenario: User has more than 15 BAT (Rewards panel)

  1. launched Brave using --enable-logging=stderr --vmodule="*/bat-native-ledger/*"=6,"*/brave_rewards/*"=6,"*/bat-native-ads/*"=6,"*/bat-native-confirmations/*"=6,"*/brave_ads/*"=9,"*/brave_user_model/*"=6 --brave-ads-staging --brave-ads-debug --rewards=staging=true,reconcile-interval=4
  2. clicked Start using Rewards and confirmed UGP token grant of 30.000 BAT
  3. clicked on the BAT icon again and clicked Verify Wallet
  4. clicked on the orange Login button and was taken to Uphold's sandbox (https://wallet-sandbox.uphold.com/)
  5. verified I could log in to Uphold, and my wallet was verified

Scenario: User has more than 15 BAT (brave://rewards page)

  1. launched Brave using --enable-logging=stderr --vmodule="*/bat-native-ledger/*"=6,"*/brave_rewards/*"=6,"*/bat-native-ads/*"=6,"*/bat-native-confirmations/*"=6,"*/brave_ads/*"=9,"*/brave_user_model/*"=6 --brave-ads-staging --brave-ads-debug --rewards=staging=true,reconcile-interval=4
  2. clicked Start using Rewards and confirmed UGP token grant of 30.000 BAT
  3. loaded brave://rewards and clicked on the Verify Wallet link
  4. clicked on the orange Login button and was taken to Uphold's sandbox (https://wallet-sandbox.uphold.com/)
  5. verified I could log in to Uphold, and my wallet was verified
example example example example example example example example
Screen Shot 2021-04-21 at 10 20 25 AM Screen Shot 2021-04-21 at 10 20 31 AM Screen Shot 2021-04-21 at 10 20 38 AM Screen Shot 2021-04-21 at 10 35 29 AM Screen Shot 2021-04-21 at 10 35 43 AM Screen Shot 2021-04-21 at 10 36 00 AM Screen Shot 2021-04-21 at 10 36 34 AM Screen Shot 2021-04-21 at 10 36 54 AM

Verification passed on

Brave 1.25.59 Chromium: 90.0.4430.212 (Official Build) beta (64-bit)
Revision e3cd97fc771b893b7fd1879196d1215b622c2bed-refs/branch-heads/4430@{#1429}
OS Ubuntu 18.04 LTS

Verified the test plan from brave/brave-core#8453

Brave Rewards Panel

image image image image

Brave Rewards page

image image image image

Verification passed on


Brave | 1.25.60 Chromium: 90.0.4430.212 (Official Build) beta (64-bit)
-- | --
Revision | e3cd97fc771b893b7fd1879196d1215b622c2bed-refs/branch-heads/4430@{#1429}
OS | Windows 10 OS Version 2004 (Build 19041.985)

UGP grants less than 15 BAT

Scenario 1: User has less than 15 BAT UGP Grants and click "Verify Wallet" via BR panel

Confirmed click on "Verify wallet" opens up "login popup" with text "15 BAT" displayed within the popup
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example Example Example Example
image image image image image image

Scenario 2: User has less than 15 BAT UGP Grants and click "Verify Wallet" via brave://rewards settings page

Confirmed click on "Verify wallet" opens up "login popup" with text "15 BAT" displayed within the popup
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example Example Example Example
image image image image image image
UGP grants =15 BAT

Scenario 1: User has 15 BAT UGP Grants and click "Verify Wallet" via BR panel

Verify wallet modal popup isn't shown after clicking on "Verify wallet" via BR panel when rewards wallet has 15 BAT - Logged #15889
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example Example Example
image image image image image

Scenario 2: User has 15 BAT UGP Grants and click "Verify Wallet" via brave://rewards settings page

Confirmed click on "Verify wallet" opens up Verify Modal popup
Confirmed click on Verify wallet in modal popup redirects to Uphold authorization page
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example Example Example Example
image image image image image image
UGP grants greater than15 BAT

Scenario 1: User has 30 BAT UGP Grants and click "Verify Wallet" via BR panel

Confirmed click on "Verify wallet" opens up Verify Modal popup
Confirmed click on Verify wallet in modal popup redirects to Uphold authorization page
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example Example Example Example
image image image image image image

Scenario 2: User has 30 BAT UGP Grants and click "Verify Wallet" via brave://rewards settings page

Confirmed click on "Verify wallet" opens up Verify Modal popup
Confirmed click on Verify wallet in modal popup redirects to Uphold authorization page
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example Example Example Example
image image image image image image
AD grants less than 15 BAT

Scenario 1: User has less than 15 BAT AD grants and click "Verify Wallet" via BR panel

Confirmed click on "Verify wallet" opens up "login popup" with text "15 BAT" displayed within the popup
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly and Ad grants are moved to brave browser uphold card
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example
image image image

Scenario 2: User has less than 15 BAT AD grants and click "Verify Wallet" via brave://rewards settings page

Confirmed click on "Verify wallet" opens up "login popup" with text "15 BAT" displayed within the popup
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly and Ad grants are moved to brave browser uphold card
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example
image image image
AD grants greater than15 BAT

Scenario 1: User has 23 BAT Ad Grants and click "Verify Wallet" via BR panel

Verify wallet modal popup isn't shown after clicking on "Verify wallet" via BR panel when the wallet balance is less than 25 BAT - logged #15889
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly and Ad grants are moved to brave browser uphold card
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example Example Example
image image image image image

Scenario 2: User has 23 BAT AD Grants and click "Verify Wallet" via brave://rewards settings page

Confirmed click on "Verify wallet" opens up Verify Modal popup
Confirmed click on Verify wallet in modal popup redirects to Uphold authorization page
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly and Ad grants are moved to brave browser uphold card
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example Example Example Example
image image image image image image
Ad grants =15 BAT

Scenario 1: User has 15 BAT AD grants and click "Verify Wallet" via BR panel

Verify wallet modal popup isn't shown after clicking on "Verify wallet" via BR panel when rewards wallet has 15 BAT - Logged #15889
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly and Ad grants are moved to brave browser uphold card
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example
image image

Scenario 2: User has 15 BAT AD Grants and click "Verify Wallet" via brave://rewards settings page

Confirmed click on "Verify wallet" opens up Verify Modal popup
Confirmed click on Verify wallet in modal popup redirects to Uphold authorization page
Confirmed click on "Login" redirect to the user to Uphold for authorization
Confirmed able to login using existing uphold credentials
Confirmed wallet balance is shown correctly and Ad grants are moved to brave browser uphold card
Confirmed user can tip KYC'd pub using uphold amount
Confirmed transactions were shown correctly in uphold dashboard

Example Example Example
image image image

@stephendonner
Copy link

@CloggyAbroad can you still reproduce this with a nightly build (https://brave.com/download-nightly/), against the staging environment, following my steps? Thanks!

@CloggyAbroad
Copy link

CloggyAbroad commented Apr 22, 2021

@stephendonner the steps you describe are primarily to verify the limit has been reduced to 15 BAT. I have no doubt this works. What I highlighted is that if you have less than 15 BAT in your rewards wallet (0.25 in my case) and you press the Rewards Logo in the address bar, followed by the Verify Wallet button, a text box pops up saying you need 25 BAT to verify. If I do the same from the brave://reward page a text box pops up saying I need 15 BAT to verify. This is what I showed in the picture I uploaded. This is still the case now (1.25.33 on W10)

@AdityaSher
Copy link

Is this live on android as well? ( Can't see it on my end)

@srirambv
Copy link
Contributor

srirambv commented May 17, 2021

Verification passed on OnePlus 6T with Android 10 running 1.25.60 x64 Beta Build

Scenario 1 : User has less than 15 BAT (rewards panel)
  • Verified when wallet has less than 15 BAT the info modal is shown when clicked on Verify Wallet
  • Verified text clearly mentions min balance of 15 BAT
  • Verified Login button is shown below the min balance message
  • Verified clicking that opens Uphold page to connect existing wallet
  • Verified able to successfully connect existing wallet and balance updates accordingly even when wallet balance is below the min threshold value
Android.ARM.15.BAT.mp4
Scenario 2 : User has exactly 15 BAT (rewards panel)
  • Verified when wallet has exact 15 BAT and clicks on Verify Wallet on rewards panel, no info message is shown
  • Verified clicking on Verify Wallet opens up uphold page
  • Verified able to successfully connect existing wallet and balance updates accordingly
Android.ARM.15BAT.mp4
Scenario 3: User has more than 15 BAT (rewards panel)
  • Verified when wallet balance is more than 15BAT clicking on verify wallet doesn't show info modal
  • Verified able to successfully connect to existing Uphold account and balance updates accordingly
Android.ARM.15.BAT.mp4

Verification passed on Samsung Tab A with Android 10 running 1.25.60 x64 Beta Build

Scenario 1 : User has less than 15 BAT (rewards panel)
  • Verified when wallet has less than 15 BAT the info modal is shown when clicked on Verify Wallet
  • Verified text clearly mentions min balance of 15 BAT
  • Verified Login button is shown below the min balance message
  • Verified clicking that opens Uphold page to connect existing wallet
  • Verified able to successfully connect existing wallet and balance updates accordingly even when wallet balance is below the min threshold value

image

Scenario 2 : User has exactly 15 BAT (rewards panel)
  • Verified when wallet has exact 15 BAT and clicks on Verify Wallet on rewards panel, no info message is shown
  • Verified clicking on Verify Wallet opens up uphold page
  • Verified able to successfully connect existing wallet and balance updates accordingly
Scenario 3: User has more than 15 BAT (rewards panel)
  • Verified when wallet balance is more than 15BAT clicking on verify wallet doesn't show info modal
  • Verified able to successfully connect to existing Uphold account and balance updates accordingly

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