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

Contract Updates #4

Merged
merged 7 commits into from
Jan 11, 2021
Merged

Contract Updates #4

merged 7 commits into from
Jan 11, 2021

Conversation

mds1
Copy link
Collaborator

@mds1 mds1 commented Nov 13, 2020

Main changes:

  • Add transfer restrictions

Other changes:

  • Removed unnecessary Minted and Burned events
  • Update from Buidler to Hardhat
  • Update code comments
  • Update linter/prettier settings

You can see the factory contract deployed and verified on Rinkeby here: (https://rinkeby.etherscan.io/address/0x4FB8Ad81754a5330165277a2655d47e99A7F598E)

@mds1 mds1 requested a review from apbendi November 13, 2020 18:50
Copy link
Collaborator

@apbendi apbendi left a comment

Choose a reason for hiding this comment

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

Looks good! One minor stylistic nitpick— nice work though :)

contracts/PToken.sol Outdated Show resolved Hide resolved
@mds1
Copy link
Collaborator Author

mds1 commented Nov 18, 2020

Before merging this, I'm going to deploy the final mainnet and Rinkeby versions. But I'm going to wait a few days to do that final deploy, because:

  • We verify the contract on Etherscan using Hardhat's Etherscan plugin
  • As you can see on our last Rinkeby Etherscan deployment, file 12 of 12 is MockDAI.sol—it included an unrelated test file as part of the verification, which we don't want
  • This is due to a bug in solc
  • Hardhat will have their own fix for this in a few days

Because Etherscan verification is a pain, and we still need to finish a few other tasks before launch, I think it's ok to wait a few days until that's fixed before deploying the final contracts

mds1 added 2 commits January 7, 2021 06:41
Removed the immutable keyword from the Factory contract, because
Etherscan would not verify the contract when present. Since the only
benefit of that keyword is slightly reduced gas costs when creating a
token, it was easiest to just remove that keyword
@mds1
Copy link
Collaborator Author

mds1 commented Jan 11, 2021

Mainnet and rinkeby contracts deployed and verified on Etherscan. Both are at 0x358bcf43fe7ec2659aD829F3604c72781fc93a9E

@mds1 mds1 merged commit e88fee2 into master Jan 11, 2021
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.

3 participants