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 Deferred Transactions from System Contract #29

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

ericpassmore
Copy link
Contributor

Change Description

Removed deferred transactions from delegate_bandwidth.cpp and name_bidding.cpp by removing the code to handle deferred transactions. As a result contracts will no longer be able to create new deferred transactions. This change is in conjunction with Block Producers preventing deferred transactions from executing. These changes will be applied to system contracts and deployed to production. The work is tracked here
eosnetworkfoundation/eos-system-contracts#86

Deployment Changes

  • Deployment Changes

EOSIO System Contract will need to be redeployed.

API Changes

  • API Changes
    No changes to API

Documentation Additions

  • Documentation Additions
    Updated documentation to remove explicit references to deferred transactions.

@ericpassmore ericpassmore changed the title rm deferred trx Remove Deferred Transactions from System Contract Sep 22, 2023
@ericpassmore
Copy link
Contributor Author

Similar changes in EOS System Contracts tracked with this PR
eosnetworkfoundation/eos-system-contracts#88

@ericpassmore
Copy link
Contributor Author

Testing

Contracts have been build using CDT 4.0 deployed from deb package, and build using leap build dir from release v4.0.4 and release tag API-early-v5.0-b73c28d51. All unit tests have passed and system contract was successfully deployed to both versions of leap specified above.

@ericpassmore ericpassmore merged commit a199ba0 into main Sep 22, 2023
@ericpassmore ericpassmore deleted the ehp/GH-28-rm-deferred-from-system branch September 22, 2023 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants