status-im / status-mobile

a free (libre) open source, mobile OS for Ethereum
https://status.app
Mozilla Public License 2.0
3.88k stars 984 forks source link

Incorrect amount populated with QR code scan of EIP-681 #12587

Closed hesterbruikman closed 1 year ago

hesterbruikman commented 3 years ago

Bug Report

Problem

When scanning a QR code to make a payment, wallet populated the amount with zeros. Maybe amount in Gwei? QR code was reportedly following eip-681

Expected behavior

When using QR code scanner tx input fields are populated correct and readible with data from url

Actual behavior

Screenshot_20210915-182451_Status

Reproduction

1) Open Status 2) Open Wallet 3) Tap QR code scanner 4) Scan QR code that follows eip-681 with $275 in DAI (not sure if issue occurs with other tokens)

Additional Information

Can send Trace logs if needed, please ping if that helps. Didn't notice anything in particular; seems like a UI bug

flexsurfer commented 3 years ago

how can we make sure QR code is correct ?

hesterbruikman commented 3 years ago

I believe this using was used: https://github.com/esPass/pretix-eth-payment-plugin

I haven't checked, but I suspect it includes QR code generation

churik commented 3 years ago

@hesterbruikman afaik we didn't use the amount converted in USD in QRs, I didn't check it because we don't have such option in wallet. I've tested always with https://brunobar79.github.io/eip681-link-generator/# and I didn't see any option to add converted value to amount. I may be wrong though, but we need QR for investigating this

churik commented 1 year ago

closing due to re-design, label recheck added for future re-consideration