Skip to content
This repository has been archived by the owner on Mar 14, 2023. It is now read-only.

Update should be opaque #55

Closed
LLFourn opened this issue Nov 1, 2022 · 1 comment
Closed

Update should be opaque #55

LLFourn opened this issue Nov 1, 2022 · 1 comment

Comments

@LLFourn
Copy link
Owner

LLFourn commented Nov 1, 2022

Originally I had the fields as pub because I thought it might be nice to be able to serialize/deserialize it. I think it's more important that we exlcude invalid states from this thing such as transactions being higher than the highest checkpoint etc.

Consider doing this at the same time as #54
@LLFourn
Copy link
Owner Author

LLFourn commented Nov 9, 2022

Fixed by using SparseChain us the update struct.

@LLFourn LLFourn closed this as completed Nov 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@LLFourn