use-token icon indicating copy to clipboard operation
use-token copied to clipboard

Add ability to override for test networks

Open sohkai opened this issue 5 years ago • 2 comments

It would be nice to have a way to configure tokenIconUrl() so that you can apply a list of overrides if you want to.

For example, how this is currently done in Finance: https://github.com/aragon/aragon-apps/blob/master/apps/finance/app/src/lib/icon-utils.js#L7.

sohkai avatar Feb 14 '20 20:02 sohkai

We should probably move this issue to the useToken() repo now. I am wondering if it’s something that we should provide as part of the hook API, rather than letting people use a wrapper function when they need this?

bpierre avatar Apr 08 '20 11:04 bpierre

Yes, agreed! I can see people writing their own hook around useToken() to specify the overrides for this! Or maybe we can move it up into an optional context provider to centralize it in one place?

Feel free to close this and move it to useToken()!

sohkai avatar Apr 08 '20 11:04 sohkai