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.

Sign up for GitHub

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

Jump to bottom

Add transferEth method to identity contract #144

Open
rdinicut opened this issue Apr 11, 2019 · 0 comments
Open

Add transferEth method to identity contract #144

rdinicut opened this issue Apr 11, 2019 · 0 comments
Assignees
@rdinicut

Comments

@rdinicut
Copy link
Contributor

execute method does not permit to transfer eth because it checks if the contract exist.
We need another method called transferEth that can be used for this use case. It should fail if the address is a smart contract. Execute should be used in that scenario
@rdinicut rdinicut added this to the 🐟 Amphitrite milestone Apr 11, 2019
@rdinicut rdinicut self-assigned this Apr 11, 2019
@rdinicut rdinicut changed the title Add transfer method to identity contract Add transferEth method to identity contract Apr 11, 2019
@lucasvo lucasvo removed this from the 🐟 Amphitrite milestone Apr 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rdinicut rdinicut

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lucasvo @rdinicut