Alberto Cuesta Cañada

Consider returning the original revert message with TransferHelper

2

TransferHelper overwrites the revert message, which makes it impossible to know whether the transaction failed because of insufficient balance, insufficient approval, or something else. The code to return the revert...

Consider adopting ERC1726 for ERC20DividendableEth

1

[ERC1726](https://github.com/ethereum/EIPs/issues/1726) Also [ERC1843](https://github.com/ethereum/EIPs/issues/1843) Also [ERC2222](https://github.com/ethereum/EIPs/issues/2222)

Fix ERC20DividendableEth for transfers.

1

Fix the issues with the supply first, please. A `mapping(address => uint) public dividendsPerTokenAdjustment` is required to keep track of imbalances when transferring tokens with owning dividends. In a transfer:...

Link repo and soldoc

3

When opening the soldoc page there is no link to go to the repo. When opening the repo in github there is no link to go to the soldoc page.

A use case I guess? https://www.techhq.io/8594/blockchain-development-tokenize-digital-assets/

I think this would be a use case, more than an example or reusable code: https://hackernoon.com/how-to-represent-physical-assets-on-a-blockchain-with-erc20-fu1yo3vkz

I think this is the earliest try for a security token. There are more recent ones. It would be nice to show we know how to implement things with them.

The latest evolution of ERC165, in that it includes a global registry of contracts and the interfaces they implement. A bit mind-bending. Obviously needs an example.

ERC777 is supposed to supersede ERC20. OpenZeppelin has an implementation. I would like to have an example of how to use it.