iron-fish / ironfish

A novel cryptocurrency focused on privacy and accessibility.
https://ironfish.network
Mozilla Public License 2.0
967 stars 574 forks source link

WalletScanner skips accounts with createdAt until createdAt sequence is reached #5652

Closed danield9tqh closed 6 days ago

danield9tqh commented 1 week ago

Summary

When an account has the following createdAt: 10 head: null

the wallet scanner should not take any action on this account until it reaches block 10. Currently the wallet scanner will still connect blocks 1-10 (without decrypting transactions) but it still takes some time and it doesn't provide any value over just connecting block 10 directly

Testing Plan

Unit tests + running local wallet tests

Documentation

Does this change require any updates to the Iron Fish Docs (ex. the RPC API Reference)? If yes, link a related documentation pull request for the website.

[ ] Yes

Breaking Change

Is this a breaking change? If yes, add notes below on why this is breaking and label it with breaking-change-rpc or breaking-change-sdk.

[ ] Yes