fuels-wallet icon indicating copy to clipboard operation
fuels-wallet copied to clipboard

Fuel wallet does not have a delete button

Open BlackEyesForThree opened this issue 9 months ago • 5 comments

Motivation

When a user needs to delete one of the sub account addresses, can this feature be added without a convenient delete button? 微信图片_20240510134613

Usage example

No response

Possible implementations

No response

BlackEyesForThree avatar May 10 '24 05:05 BlackEyesForThree

Would like to help!

ameeetgaikwad avatar May 26 '24 12:05 ameeetgaikwad

Thank you very much, so we can start adding the delete button. Used to delete excess addresses from wallet users and add secondary confirmation.

BlackEyesForThree avatar May 31 '24 06:05 BlackEyesForThree

@BlackEyesForThree should i hide it when asked to delete?

Or should i remove the account from the accounts array from useAccounts ?

ameeetgaikwad avatar Jun 17 '24 10:06 ameeetgaikwad

@ameeetgaikwad It depends on the action; Deletion is only possible if the account is imported from a private account. For seed phrases, delete does not exist as accounts are just indexes, and this is why we use the wording hide account.

I think the button should be conditional; if it is from a seed phrase, we show hidden accounts. It it is from privateKey, show delete account.

luizstacio avatar Jul 04 '24 19:07 luizstacio

@ameeetgaikwad It depends on the action; Deletion is only possible if the account is imported from a private account. For seed phrases, delete does not exist as accounts are just indexes, and this is why we use the wording hide account.

I think the button should be conditional; if it is from a seed phrase, we show hidden accounts. It it is from privateKey, show delete account.

got it!

ameeetgaikwad avatar Jul 12 '24 03:07 ameeetgaikwad