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

Plugin Compatibility Fixes for Nodeos 1.8.x #14

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

trakout
Copy link

@trakout trakout commented Nov 5, 2019

Fixed:

  • (Related Issue)
  • After much trial & error, this fixes compilation issues when building this plugin with the latest stable release of Nodeos.
    • receiver is now located directly inside action (action.receipt.receiver -> action.receiver)
    • commented out the iteration on inline traces, these should all be flat transaction traces now
    • renamed accepted_block_conn to accepted_block_connection, and applied_tx_conn to applied_transaction_connection
    • Updated function signature for the lambda in chain.applied_transaction.connect

Tested on:

  • Ubuntu 18.04 (fresh install)
  • Nodeos 1.8.5

trakout and others added 3 commits October 23, 2019 21:29
* receiver is now located directly inside action (action.receipt.receiver -> action.receiver)
* removed inline traces iteration, these should all be transaction traces now
1.8.0 compatibility attempt #1
Rename scoped connections..
don't specify a null pointer
virtual sanity check
another naming update (referencing state_history_plugin)
Update watcher_plugin.cpp
fix applied_transaction function match
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.

None yet

2 participants