-
Notifications
You must be signed in to change notification settings - Fork 160
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
burn securities #99
burn securities #99
Conversation
contracts/tokens/SecurityToken.sol
Outdated
event LogGranularityChanged(uint256 _oldGranularity, uint256 _newGranularity); | ||
event LogModuleRemoved(uint8 indexed _type, address _module, uint256 _timestamp); | ||
event LogModuleBudgetChanged(uint8 indexed _moduleType, address _module, uint256 _budget); | ||
event Mint(address indexed to, uint256 amount); | ||
event Burn(address indexed _burner, uint256 _value); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we change event names to:
Minted
Burnt
this keeps naming consistent (past tense) and differentiates a bit from the same function names.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe also move these events into IST20.sol?
contracts/tokens/SecurityToken.sol
Outdated
|
||
function burn(uint256 _value) checkGranularity(_value) public { | ||
require(tokenBurner != address(0), "Token Burner contract address is not set yet"); | ||
require(_value <= balances[msg.sender], "Value should no be greater than the balance of meg.sender"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
msg.sender rather than meg.sender
No description provided.