67P / kredits-contracts

:warning: [MOVED] Smart contracts and JS API for Kosmos Kredits
https://gitea.kosmos.org/kredits/contracts
4 stars 4 forks source link

Get rid of warnings on Kredits initialization #150

Closed raucao closed 2 years ago

raucao commented 5 years ago

Every kredits client displays 5 rows of warnings when it initalizes a new Kredits prototype:

WARNING: Multiple definitions for newAppProxy
WARNING: Multiple definitions for newAppProxyPinned
WARNING: Multiple definitions for newAppInstance
WARNING: Multiple definitions for newPinnedAppInstance
WARNING: Multiple definitions for canPerform
bumi commented 5 years ago

I looked at that awhile ago but did not find a solution. I think it is related to ethers.js

raucao commented 5 years ago

Example from ipfs-pinner:

Screenshot from 2019-07-02 18-45-56

raucao commented 5 years ago

Found the code in ethers (you can search for "Multiple definitions" in there). We'd have to do a PR to maybe change the log level it's using for those warnings (to debug instead of warn).