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

Remove GenDelegs and IRewards from the state #4620

Open
lehins opened this issue Sep 10, 2024 · 0 comments
Open

Remove GenDelegs and IRewards from the state #4620

lehins opened this issue Sep 10, 2024 · 0 comments
Labels
💳 technical-debt Issues related to technical debt we introduced

Comments

@lehins
Copy link
Collaborator

lehins commented Sep 10, 2024

We should have at least dropped contents of those fields during translation to Conway, but they aren't that big, so it is not really a problem. In fact Irewards should be empty anyways and GenDelgs should contain a Map with 7 elements in it.

That beind said, those concepts have been removed in Conway, so there is no reason for them to be hanging around in the ledger state going forward.

The proper fix for this is to move those fields into some part of the ledger state that has been changed in Conway (like GovState). If that is not possible we can figure out a new field in the ledger state that these guys could migrate to for pre-Conway eras.

@lehins lehins added the 💳 technical-debt Issues related to technical debt we introduced label Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💳 technical-debt Issues related to technical debt we introduced
Projects
None yet
Development

No branches or pull requests

1 participant