search

trufflesuite / drizzle

Reactive Ethereum dapp UI suite
906 stars 238 forks source link

Incompatibility with Solidity 0.6 ABI's #74

Closed nicholaspai closed 4 years ago

nicholaspai commented 4 years ago

0.6 ABI's are slightly different from 0.5. For example, methods in 0.6 no longer have constant or payable properties in the JSON.

One consequence is that useCacheSend hooks don't work (resulting in errors at run-time: f.cacheSend is not a function)

Initial thread on Spectrum

nicholaspai commented 4 years ago

@cds-amal : In 0.6, you can replace checking for whether a function is constant with whether its stateMutability property is view or pure. Looks like you can change this in the store package in src/contractStateUtils, src/DrizzleContract and types/IContract.d.ts.

docs

ghost commented 4 years ago

Hi there, @MalloryLeeWong and I are working on a PR that incorporates this and additional changes to make it easier to test for changes going forward. We also detected an edge case with the ABI spec that requires more testing.

The timeline is EOW or beginning of next week latest.

nicholaspai commented 4 years ago

Just wanted to check in here and see if any updates? Looks related to #80

cds-amal commented 4 years ago

Hey @nicholaspai. Apologies for the lack of response. Things have been hectic on all fronts. I'll get the ball rolling this week.

LorenzoZaccagnini commented 4 years ago

any update on when the xMNG PR will be merged into master?

fakel commented 3 years ago

any update on when the xMNG PR will be merged into master?

I tried this PR for use with the vue-plugin and it seems to work properly.