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

0.3.0 #2

Open
wants to merge 10 commits into
base: 0.3.0
Choose a base branch
from
Open

0.3.0 #2

wants to merge 10 commits into from

Conversation

j4ys0n
Copy link

@j4ys0n j4ys0n commented Apr 11, 2019

need to include compiled module and typings to use with npm install (at least in typescript). otherwise when running tsc in a project including this module, it can't find what it needs.

@Mrtenz
Copy link

Mrtenz commented Apr 11, 2019

The compiled code and typings are included in the npm version. There's no need to publish this to GitHub too (and is actually not recommended).

@j4ys0n
Copy link
Author

j4ys0n commented Apr 11, 2019

got it. it wasn't working right when i ran a fresh npm install (couldn't find types in lib) and then i wanted to install your 0.3.0 version and couldn't do so through npm.

trying to make abi work.

@j4ys0n
Copy link
Author

j4ys0n commented Apr 11, 2019

getABI is now working pretty well for functions and events. I think we need argument names in there (if that's possible), at least for events, to be able to use this to get event logs with web3. Need to add mappings and variables and ABI should be fairly complete.

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

Successfully merging this pull request may close these issues.

2 participants