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

Add default account option #289

Closed
iamdefinitelyahuman opened this issue Dec 31, 2019 · 0 comments · Fixed by #391
Closed

Add default account option #289

iamdefinitelyahuman opened this issue Dec 31, 2019 · 0 comments · Fixed by #391
Labels
enhancement New feature or request network Related to brownie.network
Milestone

Comments

@iamdefinitelyahuman
Copy link
Member

Currently if you attempt to deploy a contract without explicitly declaring from, brownie raises AttributeError: Contract has no owner, you must supply a tx dict with a 'from' field as the last argument.

  1. This error is confusing. It's meant to be given from an already deployed contract.
  2. It should be possible to declare a default account, similar to web3.eth.defaultAccount, that contracts will be deployed from. Brownie already has the default_contract_owner config setting, so perhaps expand on that?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request network Related to brownie.network
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant