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

update transaction to dawn-v3.0.0 #49

Merged
merged 1 commit into from
Apr 18, 2018
Merged

Conversation

tbfleming
Copy link
Contributor

@tbfleming
Copy link
Contributor Author

I forgot about one of the repos. This also depends on EOSIO/eosjs-api#5 .

@jcalfee
Copy link
Contributor

jcalfee commented Apr 10, 2018

Cool! thanks .. I'm looking into getting a dawn3 in docker. Right now the daily images may or may not be the exact release. When I get that going this will be easy to check and merge.

@tbfleming
Copy link
Contributor Author

I forgot to check the C++ serializer's naming before making the changes. It serializes fc::unsigned_int as "varuint32".

@tbfleming
Copy link
Contributor Author

@jcalfee I'm working on another fix to fcbuffer (it tries to use Node's Buffer operations on a Uint8Array I pass in). The problem is it's on top of this unrelated fix since that's the only way I can test. How should I produce the pull request? Create a new one that also pulls in the dawn 3 fix? Add it to the dawn 3 pull request?

@jcalfee jcalfee mentioned this pull request Apr 13, 2018
@jcalfee jcalfee merged commit d73cfd2 into EOSIO:master Apr 18, 2018
@jcalfee
Copy link
Contributor

jcalfee commented Apr 18, 2018

merged in #52

tbfleming pushed a commit that referenced this pull request Sep 28, 2018
metamaster2021 pushed a commit to armoniax/eosjs that referenced this pull request May 7, 2022
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.

2 participants