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

Change to another wallet while syncing - The wallet file is blocked sometimes. #610

Closed
juanky201271 opened this issue Mar 8, 2024 · 1 comment · Fixed by #616
Closed
Labels
enhancement New feature or request

Comments

@juanky201271
Copy link
Contributor

Only sometimes, when you try to change to another wallet and the App is syncing, the App is not able to remove the wallet file, likely because the App is writing the wallet file & is blocked when it try to delete it.

I experienced a couple of times in production, and the user can be upset about this, obviously.

This problem is really similar to that: #601

I think can be a good idea to run the interrupt_sync.... to true when the user start the action of Change to another server, so when the user confirm that, the syncing hopefully is finished, and the wallet file will be remove with no issues.

@Edicksonjga I know you experienced this as well, feel free to comment anything you want about it.

@juanky201271 juanky201271 added the enhancement New feature or request label Mar 11, 2024
@juanky201271 juanky201271 changed the title Chango to another wallet while syncing - The wallet file is blocked sometimes. Change to another wallet while syncing - The wallet file is blocked sometimes. Mar 12, 2024
@juanky201271 juanky201271 linked a pull request Mar 27, 2024 that will close this issue
juanky201271 added a commit to juanky201271/zingo-mobile that referenced this issue Mar 27, 2024
@juanky201271
Copy link
Contributor Author

Fix: The app stop the syncing process right before the App is removing the actual wallet file, this way nothing can blocked this file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant