-
Notifications
You must be signed in to change notification settings - Fork 215
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
feat(core): new output features for changes in contracts #4169
feat(core): new output features for changes in contracts #4169
Conversation
…c-change-features
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.
Looks good - utAck (most code changes leverage type safety + CI tests) - some minor optimisations in the conversion code but happy to approve
applications/tari_app_grpc/src/conversions/sidechain_features.rs
Outdated
Show resolved
Hide resolved
applications/tari_app_grpc/src/conversions/sidechain_features.rs
Outdated
Show resolved
Hide resolved
applications/tari_app_grpc/src/conversions/sidechain_features.rs
Outdated
Show resolved
Hide resolved
@sdbondi I implemented the improvements on conversions that you suggested |
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.
👍
…c-change-features
Description
SidechainFeatures
struct.CommitteeSignatures
to wrap a list of signatures, in a similar fashion to the existingCommitteeMembers
.Motivation and Context
We need a way to represent, in UTXO features, all the information needed to specify changes in a contract:
ContractUpdateProposal
: specifies the new proposed constitution.ContractUpdateProposalAcceptance
: acceptance, by a validator node, of a contract update proposal.ContractAmendment
: ratification of a update proposal, specifying when it's going to be enforced in the base layer.How Has This Been Tested?