zkBob / zkbob-ui

Apache License 2.0
3 stars 2 forks source link

Payment link improvements #223

Closed maikReal closed 11 months ago

maikReal commented 1 year ago

After the conversation with users, I got several things that we can improve:

Design - https://www.figma.com/file/VgEAclPmc6tghBwaSfTWAn/zkBob-App?type=design&node-id=4240%3A80510&mode=design&t=pNDoswLChYLJejyZ-1

1️⃣ Add the amount of available tokens on the wallet to the list of tokens

We need to show how much of tokens a user has in his wallet. The amount should be shown at the right side on the token on the list of tokens

Image

2️⃣ Add the confirmation about the completed operation

A user needs to get a receipt for the payment that he processed. So, after the payment there will be a screen with payment info + ability to download the file with this information I'll prepare the design with Tanya Statement should contain the same info that we show on the confirmation screen + txn hash and the link to the operation:

3️⃣ Change the result screen

To the result screen we need to add the statement button that user can user to download the receipt about the operation.

Screenshot 2023-09-19 at 17 00 45

4️⃣ Change USD to ETH token on OP network only

We need to make the main currency ETH, not USD. So:

5️⃣ Add notification that a user received a payment

When a user receives a payment we show the red dot on the history tab. But users don't see it. They suggested showing some notifications about the received tokens. We can reuse the same notification that we're using for errors The message should be simple: "You got X . Check your History" Related task:

6️⃣ Add the label that a transaction was from the payment link

We already have this task - https://github.com/zkBob/zkbob-ui/issues/219 UPD 12.09: Changes for the task are on the lib beta version. They aren't ready for the UI yet

maikReal commented 1 year ago

@maxaleks Could you please score this task and add your estimations?

maikReal commented 1 year ago

@maikReal Add the task for balance auto-update

vadimnikonov1 commented 11 months ago

QA review - OK

maikReal commented 11 months ago

@maxaleks With released task should be the same. Please, move tasks if it's necessary