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

Generate boilerplate or no code deploy common contracts #77 #78

Merged
merged 10 commits into from
Mar 13, 2019

Conversation

r-gochain
Copy link
Contributor

@r-gochain r-gochain commented Mar 6, 2019

Closes #77

@r-gochain r-gochain requested a review from jmank88 March 6, 2019 21:56
@r-gochain
Copy link
Contributor Author

@jmank88 thoughts?

Copy link
Contributor

@treeder treeder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use openzeppelin contacts, and preferably pull them directly from their repo. Better yet do the same as this: https://medium.com/gochain/the-easiest-way-to-deploy-an-openzeppelin-smart-contract-no-truffle-required-d248d4c3835a

The only thing to generate is that gist.

@treeder
Copy link
Contributor

treeder commented Mar 7, 2019

To expand on that, like the blog post, this should probably do the git clone part of that and things to setup the users environment to make mods and do what they want with other openzeppelin features if they want.

@r-gochain
Copy link
Contributor Author

@treeder updated, could you take a look

abi/erc20.go Outdated Show resolved Hide resolved
abi/erc20.go Outdated Show resolved Hide resolved
abi/erc20.go Outdated Show resolved Hide resolved
cmd/web3/main.go Outdated Show resolved Hide resolved
cmd/web3/main.go Outdated Show resolved Hide resolved
assets/erc20.go Outdated Show resolved Hide resolved
assets/erc20.go Outdated Show resolved Hide resolved
Copy link
Contributor

@jmank88 jmank88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good

cmd/web3/main.go Outdated Show resolved Hide resolved
assets/erc20.go Outdated Show resolved Hide resolved
@r-gochain
Copy link
Contributor Author

@treeder @jmank88 guys please check erc721 code and I'll merge

Copy link
Contributor

@jmank88 jmank88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@r-gochain r-gochain merged commit 3aa941c into master Mar 13, 2019
@r-gochain r-gochain changed the title [WIP]Generate boilerplate or no code deploy common contracts #77 Generate boilerplate or no code deploy common contracts #77 Mar 13, 2019
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.

3 participants