Global contracts: implement action for using global contract #12716
Comments
This was referenced Jan 10, 2025
github-merge-queue bot
pushed a commit
that referenced
this issue
Jan 27, 2025
Change `Account` to be enum and use different representations for serde and borsh ser/deser: * `SerdeAccount` is introduced for serde to maintain backward and forward compatibility. Previously `Account` struct was directly annotated to support serde, we are opting out of this. * `BorshVersionedAccount` is introduced for borsh serialization. Accounts in old format are serialized directly as `AccountV1`. Note that we continue serializing in old format when possible to avoid 16 bytes overhead of sentinel padding. `Account::SERIALIZATION_SENTINEL` hack is preserved for borsh serialization as I could not find a better way to handle backward compatibility. This is part of #12716, in a separate PR global contracts related fields will be added as part of Account.
Merged
github-merge-queue bot
pushed a commit
that referenced
this issue
Jan 30, 2025
There is ongoing work on [global contracts](#12716) that addresses the problem near/NEPs#491 was originally designed to solve. NEP-491 code is quite messy and has been around for about a year. We no longer see external interest in this feature, but if the need arises in the future, we can always restore the code. Reverts most of (because some part is used by #12794): - #9600 - #10701 Also, #11005 can be closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Update Account to support global contracts.
See
UseGlobalContractActionas part of prototype implementation.Both code hash and account id should be supported.
Witness contracts distribution also should be updated, see
record_contract_callThis requires changing account structure which is not trivial, but supported as part of
NEP-491implementation.Important implementation note: we need to make sure that this properly integrates with contract preparation pipeline.
The text was updated successfully, but these errors were encountered: