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

[CENT-392] write README for MINT p0 #229

Merged
merged 2 commits into from
Oct 22, 2018

Conversation

mirathewhite
Copy link
Contributor

The file doc/masterminter.md is a README file explaining the usage and functionality of the MasterMinter contract.

@@ -0,0 +1,101 @@
# MasterMinter contract
The MasterMinter is a governance contract. It delegates the functionality
of the `masterMinter` role in the Centre USDC contract to multiple addresses.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we always use the upper case of CENTRE.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

contract to call minter management functions on the FiatToken contract:

- `configureMinter(minter, allowance)` - Enables the `minter` and sets its minting allowance.
- `removeMinter(minter)` - Disables the `minter` and sets its mintingallowance to 0.
Copy link
Contributor

Choose a reason for hiding this comment

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

"mintingallowance" looks like a typo.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

The private keys to the `AllowanceController` and `SecurityController` should stay in cold storage.
This configuration lets the Controller keep multiple warm `incrementMinterAllowance` transactions on hand,
as well as the `removeMinter` transaction in case of a problem. Broadcasting the `removeMinter` transaction
will cause all future `incrementMinterAllowance` transactions to `throw`. Since there two types
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like there should be "the"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@mirathewhite
Copy link
Contributor Author

Fixed all typos detected by @o-a-hudson

@o-a-hudson o-a-hudson merged commit 1e8064c into circlefin:multi-issuer Oct 22, 2018
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