tokencard / contracts

The Consumer Contract Wallet
GNU General Public License v3.0
94 stars 40 forks source link

Controller revert message #575

Closed i-stam closed 4 years ago

i-stam commented 4 years ago
i-stam commented 4 years ago

why onlyAdmin()can claim and not onlyAdminOrOwner()?

i-stam commented 4 years ago

In general, the revert strings were chopped because they increase the binary size significantly and it was a way to avoid the wallet-deployer deployment error we were getting. I noticed that in the other statements there are articles but I think we can edit all those when V4 is merged and we have space to make them grammatically correct too. I don't have strong opinions on that, I left it up to you guys. :)