Brett Sun
Brett Sun
Hmm, the problem with just throwing an error is that most clients of the wrapper react on one of the observables firing. Perhaps we could add more information to the...
aragonUI now has a [`cachedMap`](https://github.com/aragon/aragon-ui/blob/0edc82734f7d3032cd13c5e58e1b3567a56d04c8/src/utils/cached-map.js) utility that implements a simple LRU cache
We should likely use the ABI to check the number of arguments expected, as implemented here: https://github.com/aragon/aragon.js/blob/master/packages/aragon-wrapper/src/index.js#L1110
@0x6431346e I think this will require modularizing the transaction pathing code so it's more readable and also much more testable.
@PascalPrecht it's definitely worth it to start a conversation, but they use a number of crypto dependencies that are quite large by themselves so the low hanging fruit might not...
Another small thing we could do is to use just a [particular submodule (in this case, `files-regular`)](https://github.com/ipfs/js-ipfs-http-client/blob/master/README.md#importing-a-sub-module-and-usage).
Ahh, it's more so because it requires `web3` as an input to the exposed class constructor.
We can think about building this in, but this has so many ramifications on the frontend functionality that I have a hard time justifying the savings in gas cost at...
Yes, we should probably update that so it points to a CHANGELOG.md in each app that documents what gets changed between contract updates!
> Right now if you want to transfer tokens, you need to use a wallet, add the token and then transfer it. This has always been the way I've told...