Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support stateMutability #65

Closed
axic opened this issue Jul 26, 2018 · 1 comment · Fixed by #172
Closed

Support stateMutability #65

axic opened this issue Jul 26, 2018 · 1 comment · Fixed by #172

Comments

@axic
Copy link

axic commented Jul 26, 2018

For close to a year now Solidity also outputs the stateMutability field for each function. This can have the following values:

  • pure
  • view
  • nonpayable
  • payable

For backwards compatibility also the field constant (set to true on stateMutability pure or view) and payable (set to true on stateMutability payable`) is present.

We'd like to phase out the constant and payable fields. For more see https://solidity.readthedocs.io/en/develop/abi-spec.html#json

It seems that abiParser.ts doesn't support it: https://github.com/Neufund/TypeChain/blob/master/lib/abiParser.ts#L47

@krzkaczor
Copy link
Member

@axic thanks for your comment. I actually really like this new approach. I will work on improving it when i ll find some time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants