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

add option to force full account rescan from genesis #4901

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

hughy
Copy link
Contributor

@hughy hughy commented Apr 17, 2024

Summary

resets account createdAt to null on all accounts if the 'full' option is set on requests to wallet/rescanAccount. this forces rescans for all accounts to start at the genesis block instead of beginning at the account birthday.

adds '--full' flag to wallet:rescan command.

Testing Plan

  • adds unit test
  • manual testing: checked account createdAt in JSON export before and after starting a rescan with wallet:rescan --full

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

resets account createdAt to null on all accounts if the 'full' option is set on
requests to wallet/rescanAccount. this forces rescans for all accounts to start
at the genesis block instead of beginning at the account birthday.

adds '--full' flag to wallet:rescan command.
@hughy hughy requested a review from a team as a code owner April 17, 2024 18:31
@hughy hughy changed the base branch from master to staging April 17, 2024 18:42
@hughy hughy merged commit 5451d00 into staging Apr 18, 2024
9 of 11 checks passed
@hughy hughy deleted the feat/hughy/full-rescan branch April 18, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants