Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Desktop] Unable to delete/remove created wallet in Crypto wallets #10999

Closed
Brave-Matt opened this issue Jul 29, 2020 · 21 comments
Closed

[Desktop] Unable to delete/remove created wallet in Crypto wallets #10999

Brave-Matt opened this issue Jul 29, 2020 · 21 comments

Comments

@Brave-Matt
Copy link

Description

Some users are unable to delete/remove a created wallet in crypto wallets. Once reaching the appropriate menu, the Save and Cancel options disappear and the user is returned to the main page, the wallet is not actually deleted.

Steps to Reproduce

  1. Go brave://wallet/.
  2. Click on your profile avatar
  3. Click on Settings
  4. Click on Contacts
  5. Click on My Wallet Accounts
  6. Select the wallet you want to view
  7. Click Edit
  8. Click Delete Account

Actual result:

User is returned to main page, deleted wallet remains.

Expected result:

Wallet/account should be removed.

Reproduces how often:

Easily (for the user)

Brave version (brave://version info)

Windows 10 64 Bit Home. Brave Version - [Version 1.11.101 Chromium: 84.0.4147.89 (Official Build) (64-bit)]:

Version/Channel Information:

  • Can you reproduce this issue with the current release? Y
  • Can you reproduce this issue with the beta channel? N/A
  • Can you reproduce this issue with the nightly channel? N/A

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

Original user report:
https://community.brave.com/t/i-cant-delete-my-2nd-wallet/146316

@ryanml mentions a likely fix in upcoming version update, when we rebase to MetaMask v8:
https://bravesoftware.slack.com/archives/CBP2ZKC0H/p1596040670372800

@Brave-Matt Brave-Matt added feature/web3/wallet Integrating Ethereum+ wallet support OS/Desktop labels Jul 29, 2020
@ryanml ryanml added the priority/P3 The next thing for us to work on. It'll ride the trains. label Jul 31, 2020
@ryanml ryanml self-assigned this Jul 31, 2020
@ryanml ryanml added this to the Ethereum Remote Client 1.0.22 milestone Jul 31, 2020
@srirambv
Copy link
Contributor

Verification passed on

Brave 1.12.114 Chromium: 84.0.4147.135 (Official Build) (64-bit)
Revision c42bd09b3f24da1698d71d3b4f57402137163566-refs/branch-heads/4147@{#1102}
OS Windows 10 OS Version 1809 (Build 17763.1397)
Component 0.1.68
  • Verified no Delete account option is shown in contacts
    image

Verification passed on

Brave 1.13.75 Chromium: 85.0.4183.69 (Official Build) beta (64-bit)
Revision 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS Linux
Component 0.1.68
  • Verified no Delete account option is shown in contacts
    image

Verification passed on

Brave 1.14.55 Chromium: 85.0.4183.69 (Official Build) nightly (64-bit)
Revision 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS macOS Version 10.15.5 (Build 19F101)
Component 0.1.68
  • Verified no Delete account option is shown in contacts
    image

@SnowmanUT
Copy link

Any fix for this as I have this problem and want to remove old metamask wallet that is hacked. Won't let me delete and is stuck hooked to uniswap so I have to use Chrome.

@srirambv
Copy link
Contributor

Fixed via brave/ethereum-remote-client@b6a9d98

@Heten-Daiya
Copy link

@srirambv This issue #10999 has reappeared for me in Brave Version 1.19.92 Chromium: 88.0.4324.152 (Official Build) (64-bit) on Windows 10 Home Version 20H2 Build 19042.804

braveWalletErrorReproduce0

braveWalletErrorReproduce1

On clicking the edit button, there's no delete button there.

braveWalletErrorReproduce2

@urbenlegend
Copy link

This is happening to me too. This issue needs to be reopened.

@unpaid
Copy link

unpaid commented Feb 17, 2021

Hello, I am using Brave Version 1.20.103 Chromium: 88.0.4324.152 (Official Build) (64-bit) on Windows 10 Pro Version 20H2 (OS Build 19042.804) and am having the same issue, however it seems I can get the delete button to appear by making the account appear as a contact, though the button does nothing other than delete the contact. I am unsure if this functionality is intentional or not, or perhaps another bug altogether, but I encountered it while trying to delete the wallet. The steps I used to reproduce this is as follows:

  1. Go to brave://wallet/
  2. Click on your profile avatar
  3. Click on Settings
  4. Click on Contacts
  5. Click on My Wallet Accounts
  6. Select the wallet you want to delete
  7. Click Edit
  8. Click Save
  9. Click on Contacts
  10. Select the wallet you want to do delete
  11. Click Edit

Screenshot 2021-02-18 043129

@bsclifton
Copy link
Member

cc: @bbondy

@srirambv
Copy link
Contributor

Added a note from @unpaid in #14141 so we can continue the conversation there. This issue was specifically to remove the delete button.

@Scobiform
Copy link

Now it is impossible to delete Accounts Wallets. They are just not showing? Is this the production stage?

@srirambv srirambv removed the feature/web3/wallet Integrating Ethereum+ wallet support label Sep 23, 2021
@ranrinc
Copy link

ranrinc commented Nov 17, 2021

I dont see any delete wallet button. Is this feature have been disable? Need to remove the wallet

@megmaltese
Copy link

Why is this closed?
I find it incredible that the easiest thing of all, to remove the wallet altogether, is not implemented.
And the issue here closed!
What the hell?

@bsclifton
Copy link
Member

This issue was for the older "MetaMask" fork version of wallet (Crypto Wallet). There is a new Wallet feature in Brave which is not an extension.

Under brave://settings/wallet you can pick None for Default cryptocurrency wallet and you should be good to go
image

cc: @jamesmudgett

@bbondy
Copy link
Member

bbondy commented Nov 23, 2021

Yes to the default cryptocurrency wallet setting mentioned above and you can also turn off the Show Brave Wallet icon on toolbar if you want to get rid of it

@anishp55
Copy link

how do we DELETE/REMOVE an existing wallet. not disable the whole extension. I am quite surprised this is still an issue after a year.

@Ayern12
Copy link

Ayern12 commented Nov 28, 2021 via email

@anishp55
Copy link

no what?

@Ayern12
Copy link

Ayern12 commented Nov 28, 2021 via email

@anishp55
Copy link

no one is deleting your wallet. this is a feature request to be able to delete a wallets inside of the brave browser.

@zvikasdongre
Copy link

Any update, guys?

@afuego
Copy link

afuego commented Jun 30, 2022

Reset the Brave wallet, make sure you back up your data.

@lumos7
Copy link

lumos7 commented Oct 26, 2023

is this still not possible?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests