Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

QR codes not being displayed when creating a new wallet #11492

Closed
kjozwiak opened this issue Oct 12, 2017 · 2 comments
Closed

QR codes not being displayed when creating a new wallet #11492

kjozwiak opened this issue Oct 12, 2017 · 2 comments

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Oct 12, 2017

Description

When creating a new wallet, you'll notice that the QR codes are not being displayed under the "Add Fund" manager.

Steps to Reproduce

  1. launch the latest brave (0.19.48 was being used in this instance)
  2. create a new wallet under about:preferences#payments
  3. select "Add Funds" and click on any cryptocurrency that's listed in the manager

Actual result:

screen shot 2017-10-12 at 1 16 59 pm

QR code isn't being displayed.

Expected result:

QR code should be visible as another way of adding funds into the wallet.

Reproduces how often: [What percentage of the time does it reproduce?]

100% reproducible using the above STR.

Brave Version

about:brave info:

Brave: 0.19.48
rev: de939f6
Muon: 4.4.28
libchromiumcontent: 61.0.3163.100

Reproducible on current live release:

No, part of BAT which is being introduced in 0.19.x

@kjozwiak kjozwiak added this to the 0.19.x (Beta Channel) milestone Oct 12, 2017
@kjozwiak kjozwiak changed the title QR Codes not appearing QR codes not being displayed when creating a new wallet Oct 12, 2017
@kjozwiak kjozwiak modified the milestones: 0.19.x (Beta Channel), 0.19.x Release 2 (Beta Channel) Oct 12, 2017
@NejcZdovc NejcZdovc self-assigned this Oct 12, 2017
@bsclifton
Copy link
Member

bsclifton commented Oct 12, 2017

I believe I found the issue

This call should be generating the code- it's never getting called:

state = generatePaymentData(state)

I believe this was broken here:
c965002#diff-3cdfdb124f718787d584e9e2b82fcedcR1720

Because the unconfirmed amount is 0 (which matches the state), it returns without making the call. Looking into a fix...

@kjozwiak
Copy link
Member Author

Also checked the following under Ubuntu 17.04 x64:

  • ensure that the QR codes are appearing when creating a new wallet with a clean profile under 0.19.52
  • ensure that the QR codes are appearing when upgrading from 0.19.36 -> '0.19.52'
  • esnure that the QR codes are appearing once a wallet as been recovered under '0.19.52'

syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.