EOS Dawn3 API support #25
Comments
I'll be updating the readme to advise users to use 2.x for now till dawn 3 is ready. |
Related: EOSIO/eos#1513 |
Related: EOSIO/eos#1514 |
EOSIO/eos#1513 got merged 🎉 :) Is there any tag where I can start testing dawn3-eosjs? |
The dawn3 branch in eosio/eosjs is still too buggy to use.. I'm working on it now. |
In master, transaction actions print with an empty data attribute #1565 |
Unit tests and README passing with github/eosio/eos[master] (pre-release of dawn3). Deployed to NPM, use a tag to get this version (7.0.0):
The default at npm is still to install dawn2 compatible with the public testnet (6.x.x). |
For examples, look at the README on the dawn3 branch: https://github.com/eosio/eosjs/tree/dawn3 |
@jcalfee I have 2 issues with this branch for newaccount function. This format now work: eos.newaccount({
creator: 'eosio',
name,
owner: {"threshold": 1, "keys": [], "accounts": [{"permission":{ "actor":"jj.test", "permission":"active" }, "weight": 1}]},
active: {"threshold": 1, "keys": [], "accounts": [{"permission":{ "actor":"jj.test", "permission":"active" }, "weight": 1}]},
recovery: {"threshold": 1, "keys": [], "accounts": [{"permission":{ "actor":"jj.test", "permission":"active" }, "weight": 1}]}
}) But when I try to use keys, I'm getting an error. My test is: eos.newaccount({
creator: 'eosio',
name,
owner: {"threshold": 1, "keys": [{"key": "EOS6ePVMSdSkGn4wDMqcCSTSN3GnRgEPxDPEioQQXUkfYxT8jrudg", "weight": 1}], "accounts": []},
active: {"threshold": 1, "keys": [{"key": "EOS6ePVMSdSkGn4wDMqcCSTSN3GnRgEPxDPEioQQXUkfYxT8jrudg", "weight": 1}], "accounts": []},
recovery: {"threshold": 1, "keys": [{"key": "EOS6ePVMSdSkGn4wDMqcCSTSN3GnRgEPxDPEioQQXUkfYxT8jrudg", "weight": 1}], "accounts": []}
}) The error I get is:
So it seems there is an issue when comparing the keys, even though they are the same. the issue seems to be because of
Changing it to the follow code fix the issue, but there is probably a more elegant solution:
Let me know if I should open a new ticket. |
@jeremi should be fixed now in v7.0.1 and on the dawn3 branch.. |
Yes, it's working now. I think the tests will need to be fixed as the format of parameters changed: https://github.com/EOSIO/eosjs/blob/dawn3/src/index.test.js#L153 I did not run those tests automatically as I'm not sure how to set up the test environment, but running this test manually, it fail. This doc is also outdated: |
Travis does not have a dawn3 node or a way to test with tokens, package.json forces testing locally before every npm publish. The test and syntax works for me though, eosjs converts that syntax into the more expanded syntax your using: see 'short-hand' in the README. Both ways should work. |
No description provided.
The text was updated successfully, but these errors were encountered: