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

ERC20 extension for governance tokens (vote delegation and snapshots) #2632

Merged
merged 44 commits into from
May 12, 2021

Conversation

Amxx
Copy link
Collaborator

@Amxx Amxx commented Apr 19, 2021

The goal is to offer an ERC20 extension that can be included in a token to make it suitable for on-chain governance with vote delegation. This extension is meant to be COMP-compatible.

  • check compatibility against current comp tokens

PR Checklist

  • Tests
  • Documentation
  • Changelog entry

@frangio frangio mentioned this pull request Apr 20, 2021
3 tasks
@frangio frangio changed the title Comp extension for ERC20 tokens ERC20 extension for governance tokens (vote delegation and snapshots) Apr 20, 2021
contracts/token/ERC20/extensions/ERC20Snapshot.sol Outdated Show resolved Hide resolved
contracts/token/ERC20/extensions/ERC20Snapshot.sol Outdated Show resolved Hide resolved
contracts/token/ERC20/extensions/ERC20Snapshot.sol Outdated Show resolved Hide resolved
contracts/token/ERC20/extensions/draft-ERC20Votes.sol Outdated Show resolved Hide resolved
contracts/token/ERC20/extensions/draft-ERC20Votes.sol Outdated Show resolved Hide resolved
contracts/token/ERC20/extensions/draft-ERC20Votes.sol Outdated Show resolved Hide resolved
contracts/token/ERC20/extensions/draft-ERC20Votes.sol Outdated Show resolved Hide resolved
@Amxx Amxx marked this pull request as ready for review May 11, 2021 16:00
@frangio frangio force-pushed the feature/erc20/comp branch from f0c641f to 5060eb9 Compare May 12, 2021 01:51
Copy link
Contributor

@frangio frangio left a comment

Choose a reason for hiding this comment

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

LGTM

@frangio frangio merged commit 100ca0b into OpenZeppelin:master May 12, 2021
@Amxx Amxx deleted the feature/erc20/comp branch May 15, 2021 08:48
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.

2 participants