Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Resolving actions for eosio.null #1009

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

o-alexandrov
Copy link

@o-alexandrov o-alexandrov commented Aug 12, 2021

Closes #814
Here is an example completed transaction after the code changes in this PR.

Change Description

There is a regression since eosjs v16 where this repo used to hardcode the eosio.null abi, instead of attempting to fetch it.
The bug is described here: #814

Or to rephrase the issue description above, the problem is:

API Changes

  • API Changes

The cachedAbi is initialized with a hardcoded abi for eosio.null.
So developers could make the transactions unique, by adding either an action or a context_free_action, example below:

context_free_actions: [
  {
    account: `eosio.null`,
    name: `nonce`,
    authorization: [],
    data: { value: `test data` },
  },
],

@o-alexandrov
Copy link
Author

@bradlhart
Could you please let me know what is missing here or how could I assist so we merge it?

@o-alexandrov
Copy link
Author

@bradlhart could you please update here?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant