change the " Deposit " Function

[sync-by-unito]

Suggest the update for user story:

As a Capture App user, when I click the "Deposit" button, I want to see QR code of my asset-wallet address with a copy button to copy the address automatically (like Binance and any exchange App) so that I can easily deposit fund from my other wallets to the Capture asset wallet.

——
the original user story:
as user , when i click the " Deposit " it will show the asset wallet address & i can copy the address ( Better to have asset wallet address qrcode displayed )

┆Issue is synchronized with this Asana task by Unito
┆Created By: Scott Yan

[sync-by-unito]

➤ Tammy Yang commented:

Suggest the update for user story:

As a Capture App user, when I click the "Deposit" button, I want to see QR code of my asset-wallet address with a copy button to copy the address automatically (like Binance and any exchange App) so that I can easily deposit fund from my other wallets to the Capture asset wallet.

[sync-by-unito]

➤ Sam commented:

Scott Yan, when I go to Wallets page and click "Deposit" it shows me this ( https://i.imgur.com/tPKINHs.png ) (there is no asset wallet address).

But when I click wallet it shows this page ( https://i.imgur.com/hgspdbY.png ) and there we have "My Asset Wallet address" ( https://i.imgur.com/wTrUTFY.png ).
So I need to show QR code instead of "My Asset Wallet address" ( https://i.imgur.com/wTrUTFY.png ) right?

[sync-by-unito]

➤ Scott Yan commented:

show QR & a copy button to copy the address

[sync-by-unito]

➤ Sam commented:

Scott Yan, here is the UI proposal ( https://i.imgur.com/7SJx5gB.png ). By the way when the user scans the QR code what is the expected behavior?

[sync-by-unito]

➤ Kenny Hung commented:

Sam cc Scott Yan I think user scans the qrcode should be like this.
Meanwhile the qr code should display in deposit page instead of wallet page.

[sync-by-unito]

➤ Sam commented:

Kenny Hung, thank you very much. Now it's 💯 clear.

[sync-by-unito]

➤ Sam commented:

Here is the pull request ( #1782 ).

[sync-by-unito]

➤ Bofu Chen commented:

PR has been merged
#1782 ( #1782 )

Sam Does this PR have been included in 0.60.0, or will it be included in 0.61.0?