Skip to content

Use openzeppelin-solidity ERC20 contract from npm

Jared Pereira requested to merge openZepplinERC20 into master

Currently we are using the ERC20 contracts from https://github.com/ConsenSys/Tokens/.

Several things motivated a switch:

  • We were manually copying and pasting them, instead of having them in some kind of package
  • They didn't implement the contract we needed without a constructor

The open-zeppelin contracts can be installed directly from NPM, and have fairly wide community adoption. Truffle allows us to import contracts installed in node_modules so this works perfectly.

Changes

This only motivated one change in our contracts, replacing totalSupply with totalSupply_

Merge request reports