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

Birthday height is not being respected as the sync start point. #823

Open
nuttycom opened this issue Sep 14, 2023 · 2 comments
Open

Birthday height is not being respected as the sync start point. #823

nuttycom opened this issue Sep 14, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@nuttycom
Copy link
Contributor

Describe the issue

When entering a wallet birthday for wallet recovery, the birthday height appears to be accepted but thereafter the wallet seems to be fully synced, even when the birthday height is millions of blocks in the past.

Can you reliably reproduce the issue?

If so, please list the steps to reproduce below:

  1. Open the debug screen
  2. Nuke wallet
  3. Restore wallet (enter seed)
  4. Enter wallet birthday

Expected behaviour

The wallet sync should proceed from the birthday height.

Actual behaviour + errors

  • App Version: TestFlight build 60

The wallet appears to be fully synced.

@nuttycom nuttycom added the bug Something isn't working label Sep 14, 2023
@nuttycom
Copy link
Contributor Author

This bug appears to only be reachable from the "nuke wallet -> restore" flow; when I nuke the wallet, then uninstall the app and reinstall, the "restore wallet" flow from the onboarding screen seems to work correctly.

@LukasKorba
Copy link
Collaborator

LukasKorba commented Sep 15, 2023

I think there is a bug somewhere else. I see wallet birthday to be set, it's passed to the Initializer + stored to the keychain. But syncing was somewhat broken when I tried these 4 steps. I suspect wipe doesn't work perfectly, not resetting some context. Will investigate it further. For me also the deletion of the app and reinstall started to work (which effectively is clean start, perfect reset that probably is not after wipe)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants