search

cake-tech / cake_wallet

The open source repository for Cake Wallet, a noncustodial multi-currency wallet, and Monero.com, a noncustodial Monero-only wallet. Need help? Check out https://guides.cakewallet.com
https://cakewallet.com
MIT License
601 stars 164 forks source link

Bug switching wallets #1484

Closed CodeCracker-oss closed 4 weeks ago

CodeCracker-oss commented 1 month ago

Describe the bug When switching from litecoin wallet to Monero wallet, it gives messages of "loading wallet" and just freezes apps, which unfreezes when clicking back or swiping message away. Never actually loading the monero wallet. I am unsure if this happens woth other wallets or not.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'menu icon'
  2. Click on 'wallets'
  3. Click on wallets name
  4. See the loading message thats loads continuously.

Platform:

Additional context Same issue occurs on prior version of Cake Wallet 4.18.0, have not tested any other version. Uninstall/Reinstall does not address issue.

BitcoinQnA commented 1 month ago

I am also experiencing the same thing on a Pixel 6a running 4.18

tuxpizza commented 1 month ago

Does this apply to a new Monero wallet you create or is it just ones from previous versions?

CodeCracker-oss commented 1 month ago

@tuxpizza I haven't tested creating a wallet with previous versions. I am new to Cake wallet, so I had just created it with v4.18.0

BitcoinQnA commented 1 month ago

@tuxpizza I'm a first time Cake user with a new Bitcoin and Monero wallet.

tuxpizza commented 1 month ago

I will run some test on my side to see if I can reproduce this, as I currently am unable to.

To be correct, this is only when switching from Litecoin to Monero? Does it occur when switching to Monero with any other wallet type if you could try?

CodeCracker-oss commented 1 month ago

@tuxpizza The issue occurs the same switching from any wallet to Monero. Does not occur when switching from Monero to another wallet.

As of right now it started working, although a delay of roughly 5 seconds before monero wallet will load.

BitcoinQnA commented 1 month ago

Here is what happens with mine. I cut the video short, but it just hangs on the loading screen.

https://github.com/cake-tech/cake_wallet/assets/64638670/6d27504a-a3ac-4537-bd95-6b62d70428e5

BitcoinQnA commented 1 month ago

@CodeCracker-oss is there any more info I can get you to assist in troubleshooting here?

CodeCracker-oss commented 4 weeks ago

@BitcoinQnA Probably not as this issue seems to have been addressed for me as I can't reproduce it on version 4.18.2 of the fdroid build. I am not the developer of Cake wallet, so I wouldn't be of much help.

tuxpizza commented 4 weeks ago

@BitcoinQnA did you try reinstalling and re-importing your seeds (Don't use the restore from backup feature in this test due to potential wallet file corruption

BitcoinQnA commented 4 weeks ago

@CodeCracker-oss my apologies, I tagged the wrong person in my previous post.

@tuxpizza this seemed to do the trick. Side question, what's the reason for you guys using the Electrum seed format instead of BIP39 like the rest of the industry?

tuxpizza commented 4 weeks ago

@CodeCracker-oss my apologies, I tagged the wrong person in my previous post.

@tuxpizza this seemed to do the trick. Side question, what's the reason for you guys using the Electrum seed format instead of BIP39 like the rest of the industry?

I'm not 100% sure why this is the way for BTC/LTC, this was a decision made by somebody years ago to use Electrum. We're actually in the process of switching to BIP-39 by default for our Bitcoin and Litecoin wallets to allow users to more easily use 1 seed for many wallets. We recently added BIP-39 restore support to those wallets, and the rest of our wallet types (Excluding monero) already use BIP-39 by default when creating a new wallet.

tuxpizza commented 4 weeks ago

Glad the reinstall fixed that for you, it must have just been some weird wallet corruption which happens in rare cases. I'll be closing this issue for now as it appears to be resolved for both of you :wink: