search

KomodoPlatform / atomicDEX

This is the official AtomicDEX mobile (smartphone app) repository
https://atomicdex.io
GNU General Public License v2.0
10 stars 15 forks source link

QR scan from Agama Invoice not working #125

Open teedan opened 4 years ago

teedan commented 4 years ago

Repro steps:

Result: The scanned information is not translated into filling the address and amount fields.

SirSevenG commented 4 years ago

Hello again. We will need a bit more info to investigate this issue.

Do you use Verus Agama wallet? What exact version?

teedan commented 4 years ago

Hi, could you reproduce it? I've just reproduced the error once again. Agama version 0.5.9-2, atomicDEX version 0.2.7 on iOS.

teedan commented 4 years ago

After scanning the QR code, the following string is put into the address field: {"address":"R...","amount":"1","coin":"KMD"} which is not accepted as a valid address.

SirSevenG commented 4 years ago

Performs similarly for me. QR scanner supposed to read QR address to send to. And it's what it does in a way - reads json in Agama QR and pastes it to Address field. We will consider to add json scans support as feature.

teedan commented 4 years ago

I had another instance with a QR code generated from an older version of the Ledger Live App (the wallet software for the Ledger Nano S hard wallet). After scanning its QR code, a string like komodo:R... was put into the address field which was also not a valid address. Not reproducable with the latest version of Ledger Live app anymore, however. But still maybe something to consider.

tonymorony commented 4 years ago

Would be great to check if it's actual with current Verus Desktop (since Agama is deprecated) and create relevant feature-request in GUI repo if it's the case @SirSevenG

SirSevenG commented 4 years ago

@tonymorony Could not reproduce on latest android version. Checked multiple QR codes generated with different apps (Verus Desktop, Atomic Wallet, AtomicDEX, etc).