perawallet / pera-wallet

Source code for Pera Wallet, simply the best Algorand wallet.
https://perawallet.app
Other
178 stars 62 forks source link

Misleading and scary CloseOut language #43

Open patrik7 opened 2 years ago

patrik7 commented 2 years ago

I am having the user sign one transaction in a group.

The group has another transaction that has CloseoutTo set. The source transaction is NOT the user's address, wallet, he's not even authorizing this transaction.

However, the Algorand wallet displays a colorful warning:

"This transaction is sending your full Algo balance"

This is very misleading, it appears as if the app was trying to steal all of the user money. The "your full Algo" balance... While in reality, we are clearing the escrow account.

Could the Algorang app check the source for the Closeout transaction before displaying such a warning to the user?

barnjamin commented 2 years ago

Seconded, this also made some folks using the decipher drop app nervous.

taylanpince commented 2 years ago

Thank you for this feedback, I am tracking this internally.