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 backwards compatilibty with ERC20 for ERC777 #1731

Closed
nventuro opened this issue Apr 29, 2019 · 1 comment · Fixed by #1735
Closed

Add backwards compatilibty with ERC20 for ERC777 #1731

nventuro opened this issue Apr 29, 2019 · 1 comment · Fixed by #1735
Labels
contracts Smart contract code. feature New contracts, functions, or helpers.
Milestone

Comments

@nventuro
Copy link
Contributor

A major feature of ERC777 is that it can be used as if it were an ERC20 token: we should implement this interface to enable this use case.

Due to ERC777's state variables being private, the most straightforward way to achieve this would be by having this backward compatibility baked-in into the contract. We could then provide a second contract that disables the ERC20 functions (by calling revert on them) for users to opt-out of this compatibility.

@nventuro nventuro added feature New contracts, functions, or helpers. contracts Smart contract code. labels Apr 29, 2019
@frangio
Copy link
Contributor

frangio commented Apr 29, 2019

I don't think we should provide the non-ERC20-compatible contract for now, it adds a lot of complexity. Let's wait and see if there is demand for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contracts Smart contract code. feature New contracts, functions, or helpers.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants