SasakiYuki / RaccoonWallet

20 stars 5 forks source link

QR読み込み時の不具合 #18

Closed SasakiYuki closed 5 years ago

SasakiYuki commented 6 years ago

baneoさんより

小数点以下4桁以降が表示されていない 送金するとされているようなので表示のみされていないという一番やな状態 直す

またメッセージも無視されている模様なのでそちらも合わせて修正する

SasakiYuki commented 6 years ago

QR形式の羅列

そもそもQR形式にどんな種類が存在するか https://github.com/SasakiYuki/RaccoonWallet/blob/3c2db2e9e2937239289a46855ae74881b55bf7f7/app/src/main/java/wacode/yamada/yuki/nempaymentapp/model/PaymentQREntity.kt

        val addr: String,
        val amount: Long,
        val msg: String,
        val name: String? = ""

上部三つがすでに入っている場合はダイレクトで決済確認画面へ飛ぶ それ以外の場合はそれぞれの場所からスタートするのが親切そう

例えばaddressのみ入っているQRなら数字入力画面から amountもすでに入っていればmsgを付けるかどうかから

nameは考慮しない(QRに含まれる場合表示だけはしてあげるが変更するUIはない)

Soma-Kato commented 6 years ago

Sasakiと議論したので、そのメモです

QRコードを読み込みの際の挙動変更

  1. ニュービーDialogの廃止->QRコードから読み込んだので、アドレスは間違ってないと思われる
  2. QRを読み込んだデータにアドレスが存在し、かつamountが0になっている場合 -> amountが0だけど大丈夫かを確認するDialogを表示 yes -> message指定画面に遷移 no -> 金額指定画面に遷移
  3. QRを読み込んだデータにアドレスが存在し、かつamountが0以上になっている場合 -> messageが空だけど大丈夫かを確認するDialogを表示 yes -> 送金確認画面に遷移 no -> message種類指定画面に遷移
  4. QRを読み込んだデータにアドレスが存在かつamountが0以上かつmessageが空じゃない場合 -> 送金確認画面に遷移 以上のように変更します