adridadou / eth-propeller-core

Core library for Ethereum propeller
Apache License 2.0
26 stars 16 forks source link

Missing support of 'signature' field in ABI #18

Closed kivanov82 closed 6 years ago

kivanov82 commented 6 years ago

Latest Solidity compiler adds new field 'signature' to the API specification. Currently this leads to the "error while deserialising ABI" exception when trying to work with the contracts.

adridadou commented 6 years ago

I was going to update the infura integration today so I can do that too. What is this field about? Do u have a link to some documentation?

Le lun. 24 sept. 2018 à 11:44, Kiril Ivanov notifications@github.com a écrit :

Latest Solidity compiler adds new field 'signature' to the API specification. Currently this leads to the "error while deserialising ABI" exception when trying to work with the contracts.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/adridadou/eth-propeller-core/issues/18, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXQiXSNxCbLGN7FQhb_PXSSYOQzf917ks5ueKlzgaJpZM4W2VuT .

kivanov82 commented 6 years ago

Cool, here is the spec: https://solidity.readthedocs.io/en/develop/abi-spec.html

Would than be possible to fix https://github.com/adridadou/eth-propeller-core/issues/16 too? It's a small change but prevents using the solidity compiler output for propeller as-is.

adridadou commented 6 years ago

I'll have a look and do my best to solve that too

Le lun. 24 sept. 2018 à 11:55, Kiril Ivanov notifications@github.com a écrit :

Cool, here is the spec: https://solidity.readthedocs.io/en/develop/abi-spec.html

Would than be possible to fix #16 https://github.com/adridadou/eth-propeller-core/issues/16 too? It's a small change but prevents using the solidity compiler output for propeller as-is.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/adridadou/eth-propeller-core/issues/18#issuecomment-423925438, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXQiZ28zgWh4MW0RxfRNNwsQOK9Z4irks5ueKwUgaJpZM4W2VuT .