liquidapps-io / zeus-sdk

EOSIO & EVM smart contract unit testing suite and API provider
51 stars 24 forks source link

Fix vAccount action payload serialization to work with big payloads #3

Closed MrToph closed 5 years ago

MrToph commented 5 years ago

The existing code to serialize vAccount actions only works with payloads that don't exceed a length of 255 bytes as it always uses a single byte for the payload size. Although this works fine in your vaccountconsumer test due to the small payload, I think it's a good idea to change it there as well because I'm sure many developers, including me, will use that code as their starting point until there's documentation.

Changelog

NatPDeveloper commented 5 years ago

Thank you @MrToph for this! I will get this code setup in our staging environment to be merged.