-
Notifications
You must be signed in to change notification settings - Fork 18
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
Move infrastructure helpers to package #185
Conversation
196a207
to
d14679e
Compare
201228e
to
583e44e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
neutronChain, | ||
daoCoreAddress, | ||
); | ||
daoParent = new dao.Dao(neutronChain, daoContracts); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe mainDao
like below?
96b476d
c8a4e09
to
e285396
Compare
@@ -39,6 +39,7 @@ | |||
"@cosmos-client/core": "0.45.13", | |||
"@cosmos-client/cosmwasm": "^0.20.1", | |||
"@cosmos-client/ibc": "^1.2.1", | |||
"@neutron-org/neutronjsplus": "^0.0.9", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the latest version is 0.0.8, isn't it? https://github.com/neutron-org/neutronjsplus/blob/e2b4bf6482a0377b996ae8de0f66af62b5a89c6c/package.json#L3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you are right, I forgot to push changes
No description provided.