Ori Pomerantz
Ori Pomerantz
I think a comment in the code will help. I'll submit a PR to that effect.
What would be the proper way to do this? Add ERC20SafetyRails.sol to https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/token/ERC20/extensions/ ?
The reason I'm trying to get this added *somewhere* is that in many cases developers don't add this feature, even though they don't need the contract to hold tokens and...
I decided to write this [as an ethereum.org article](https://ethereum.org/en/developers/tutorials/erc20-with-safety-rails/) instead.
How about this solution? If the complier sees `assert` inside a conditional statement it: 1. Outputs a warning that having `assert` inside a conditional causes inefficient code and is best...
One requested feature was to have the logs entries available in a test (https://github.com/ethereum/tests/issues/523). This would take care of that.
I don't assign tasks. I wrote the issue because I wanted the people who run the site (@minimalsm) to approve it before I write. However, if you want to write...
Articles are typically written in markdown, and submitted as a PR to this repository (under https://github.com/ethereum/ethereum-org-website/tree/dev/src/content/developers/tutorials).
For some reason github didn't close this, but since the PR is merged, it should be.
New style reverts seem to be supported on Goerli (see [the contract](https://goerli.etherscan.io/address/0x6d86ae3e08960f04932ec8e38c5ac692351114ba#code) and the [tx with a new revert](https://goerli.etherscan.io/tx/0xa37c446496073baac32c439257e133ccec965333eda633f9987a336dea4f4c64)). This is different from the behavior on Optimistic Kovan ([see tx...