-
Notifications
You must be signed in to change notification settings - Fork 12
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 to ship definitions get_blocks v1 (for fetch_finality_data) #28
Conversation
uint32_t active_finalizer_policy_generation = {}; | ||
eosio::checksum256 action_mroot = {}; | ||
eosio::checksum256 base_digest = {}; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this include proposed_finalizer_policy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I built this from
https://github.com/AntelopeIO/spring/blob/8f65101b071768e8f213cd62e1a580f55aed882b/libraries/state_history/abi.cpp#L576-L581
which is lacking proposed_finalizer_policy
. It does seem like this is an oversight in spring's abi.cpp
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does look like that was missed.
https://github.com/AntelopeIO/spring/blob/3133bc1713a992139494eb3250c94a48336566a1/plugins/state_history_plugin/state_history_plugin.cpp#L290-L290
packs finality_data_t
which contains proposed_finalizer_policy
updated and ready for re-review |
Message was added in AntelopeIO/leap#2349 and tweaked in AntelopeIO/leap#2381