openzeppelin-contracts
openzeppelin-contracts copied to clipboard
✍️ IERC20: use `amount` instead of `value`
✍️ use amount instead of value for event params to make match underlying functions
- [x] Tests
- [ ] Documentation
- [ ] Changelog entry
Hello @z0r0z
The EIP uses value and not amount. I agree that amount would have been a better choice of words, but so far we favored consistency with the EIP wording.
@frangio any opinion?
I think this PR is pointing out that the events and the functions use a different word for the same concept. Perhaps we should use "value" instead of "amount", but I would agree that it should be consistent.
The ERC20 implementation uses "amount". Should we change it all to "value"?
Closed by #4398