iron-fish / ironfish

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

removes hash from account createdAt #5043

Closed hughy closed 3 weeks ago

hughy commented 3 weeks ago

Summary

sets createdAt sequence to 'head.sequence - confirmations' to mitigate issues with setting account createdAt around the time of a chain reorg.

adds migrations 33 to migrate existing account.createdAt values to use the existing createdAt.sequence - confirmations

updates account fixtures

Testing Plan

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