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

Faucet fix #333

Merged
merged 8 commits into from
Oct 12, 2018
Merged

Faucet fix #333

merged 8 commits into from
Oct 12, 2018

Conversation

maxsam4
Copy link
Contributor

@maxsam4 maxsam4 commented Oct 11, 2018

Limited the amount of tokens that can be minted in one transaction by the faucet to 1M.

contracts/mocks/PolyTokenFaucet.sol Outdated Show resolved Hide resolved
@satyamakgec
Copy link
Contributor

@maxsam4 one thing is still left could you make the totalSupply divisible. I mean to multiply the totalSupply with 10^decimals.

@satyamakgec
Copy link
Contributor

@maxsam4 why the lockupVolumeRestricitonTM old commits are in this PR?

@maxsam4
Copy link
Contributor Author

maxsam4 commented Oct 12, 2018

@satyamakgec This branch is based on an older version of dev branch which did not have the lockupVolumeRestricitonTM changes. The last automatic merge (d8a6db8) should have deleted the extra file but it did not for some reason.
None of my commits added the old file. GitHub merge just didn't delete it for some reason. I'll delete it manually.

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