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

New on boarding for wallet verification #5518

Closed
NejcZdovc opened this issue Aug 2, 2019 · 5 comments · Fixed by brave/brave-core#3081
Closed

New on boarding for wallet verification #5518

NejcZdovc opened this issue Aug 2, 2019 · 5 comments · Fixed by brave/brave-core#3081

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Aug 2, 2019

We would like to simplify verify screen.

Things to do:

  • remove on boarding in the panel. Clicking on the button should open rewards page and open on boarding dialog
  • move panel overlay into modal which should look like this:

image

@NejcZdovc NejcZdovc added this to the 0.70.x - Nightly milestone Aug 2, 2019
@NejcZdovc NejcZdovc self-assigned this Aug 2, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Aug 2, 2019
@bsclifton
Copy link
Member

Looking at this - it's a little bit confusing and also regresses on past capability (2 way wallet regresses capability, not this text change)

For example, if I want to add funds without "verifying"... that isn't possible. The closest I can do is:

  1. Go to brave://rewards
  2. Click Add funds
  3. Click Verify Wallet (even though I don't intend to verify)
  4. Create an Uphold account
  5. Don't complete the KYC process
  6. Get the address inside Uphold
  7. Send the funds there

The previous flow was:

  1. Go to brave://rewards
  2. Click Add funds
  3. Addresses are shown where you can send Eth, BAT, etc
  4. Send the funds there

It would be nice if the addresses were shown on the screen above, for people that don't even want to click Verify Wallet (because they have no intention of verifying and don't even want to create an Uphold account)

@bradleyrichter
Copy link

@bsclifton Can you expand on the confusing aspect?

Regarding adding crypto via wallet addresses, this now requires an uphold account (not KYC) but there isn't currently a separated logical flow that can be linked to from our ad-funds button.

@bsclifton
Copy link
Member

bsclifton commented Aug 2, 2019

@bradleyrichter quick example - I have an account with BAT already where I can load up my wallet. Let's say I have 0 interest in signing up for Uphold; I purely want a one-way wallet where I put the funds, they can be dispersed. We don't really offer that anymore

Instead, you now (even to get your address) need to create an Uphold account. In our text, we say "Verify" even though you can use the Uphold account without finishing KYC (and then you have a 1 way Uphold wallet). I'd imagine some folks (as users) have no interest in verifying but would be OK with an Uphold wallet

Publishers can be required to do KYC because they are getting incoming tokens. But for you to disperse tokens, it seems a little odd (to me) to have messaging "requiring" the KYC (especially since you can just not finish the KYC process and you can still have tokens go outbound)

Example use-case:

  1. I have a CoinBase account with 1,000 BAT
  2. Every month or so, I put 100 BAT into my Brave one-way wallet. I get the address by clicking add funds.
  3. 0.69.x arrives
  4. I click add funds; address is not shown
  5. I am shown above modal and only button present is Verify Wallet. I have 0 interest in verifying, let alone creating an Uphold account
  6. I'm stuck now, not being able to transfer my BAT
  7. I can presumably tip / etc until my existing balance (for prior 1 way wallet) runs out

@bradleyrichter
Copy link

@NejcZdovc I had a typo in that screen above. Please use "necessary" instead of "necessarry".

image

NejcZdovc added a commit to brave/brave-core that referenced this issue Aug 7, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Aug 12, 2019
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Aug 22, 2019

Verification passed on

Brave 0.69.113 Chromium: 76.0.3809.100 (Official Build) beta (64-bit)
Revision ed9d447d30203dc5069e540f05079e493fc1c132-refs/branch-heads/3809@{#990}
OS Windows 10 OS Version 1803 (Build 17134.523)
  • Verified the test plan from Adds new on boarding flow brave-core#3081
  • Verified that verify modal displayed in a new tab when Verify Wallet button is clicked from BR panel
    image
  • Verified that verify modal displayed again when Verify Wallet button is clicked from brave://rewards page
  • Verified that uphold page is opened when VERIFY WALLET button is clicked on verify modal
  • Verified the verify modal display when clicking on Add Funds in BR panel
  • Verified the verify modal display when clicking on Add Funds in brave://rewards page
  • Verified the verify modal display when clicking on Withdraw Funds in brave://frewards page

Verified passed with

Brave 0.69.113 Chromium: 76.0.3809.100 (Official Build) beta (64-bit)
Revision ed9d447d30203dc5069e540f05079e493fc1c132-refs/branch-heads/3809@{#990}
OS Mac OS X

Screen Shot 2019-08-22 at 6 04 25 PM

  • Verified test plan from: Adds new on boarding flow brave-core#3081
  • Verified that the new modal is being displayed whenever clicking on the following buttons/properties under brave://rewards/ on new (0.69.x) and upgrade (from 0.68.x) profiles:
    • Verify Wallet
    • Add Funds
    • Withdraw Funds
    • (verify) <-- next to the `Brave wallet is powered by Uphold
  • Verified that the Rewards page is opened and the new modal is being displayed whenever clicking on the following buttons/properties on the rewards panel (from BAT logo) on new (0.69.x) and upgrade (from 0.68.x) profiles:
    • Verify Wallet
    • Add Funds
  • Verified that pressing "X" closes the modal without any issues
  • Verified that clicking on Verify Wallet opens uphold.com in the same tab
  • Verified that once you press Verify Wallet within the modal and login to Uphold, modal doesn't get displayed for the second time.
  • Encountered and logged URL doesn't update after closing Verify modal #5771 and unable to disconnect from Uphold if KYC is not completed #5772

Verification passed on

Brave 0.69.117 Chromium: 76.0.3809.100 (Official Build) beta (64-bit)
Revision ed9d447d30203dc5069e540f05079e493fc1c132-refs/branch-heads/3809@{#990}
OS Ubuntu 18.04 LTS

image

  • Verified test plan from: Adds new on boarding flow brave-core#3081
  • Verified that the new modal is being displayed whenever clicking on the following buttons/properties under brave://rewards/ on new (0.69.x) and upgrade (from 0.68.x) profiles:
    • Verify Wallet
    • Add Funds
    • Withdraw Funds
    • (verify) <-- next to the `Brave wallet is powered by Uphold
  • Verified that the Rewards page is opened and the new modal is being displayed whenever clicking on the following buttons/properties on the rewards panel (from BAT logo) on new (0.69.x) and upgrade (from 0.68.x) profiles:
    • Verify Wallet
    • Add Funds
  • Verified that pressing "X" closes the modal without any issues
  • Verified that clicking on Verify Wallet opens uphold.com in the same tab
  • Verified that once you press Verify Wallet within the modal and login to Uphold, modal doesn't get displayed for the second time.

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