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

TokenVault: Implementing full EIP-20 support and improved tests #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

TokenVault: Implementing full EIP-20 support and improved tests #175

wants to merge 1 commit into from

Commits on Feb 15, 2019

  1. TokenVault: Implementing full EIP-20 support and improved tests 

    Previously we had a check for isToken() interface function.
The check is important for testing if the token is correct.
However, not all of the EIP-20 tokens support such a check.
Therefore, modified the check to check totalSupply instead.

0 supply is permitted, for example the token could be minted afterwards.

Also, there were no tests for this check, added those.
    @villesundell
    villesundell committed Feb 15, 2019
    Configuration menu
    Copy the full SHA
    70c62c0 View commit details
    Browse the repository at this point in the history