uncaught at getAccounts Error: Please pass numbers as strings or BigNumber objects to avoid precision errors. about drizzle-legacy HOT 8 CLOSED

@adrianmcli Will do.

from drizzle-legacy.

Did you fix this?

from drizzle-legacy.

Think this needs a bit more clarification. The error is happening because somewhere the code is passing a value into the web3.utils.fromWei function as a number type instead of a String or BigNumber. Are you using the fromWei function in your own client-side code at all?

from drizzle-legacy.

Will attempt to reproduce on our end.

from drizzle-legacy.

I can reproduce on drizzle-box. This happens under the following scenario:

Metamask 5.3.0
drizzle-box v1.0.0

1. Login to Metamask using account 1.
2. Switch to account 2.
3. Observe the error Uncaught Error: Please pass numbers as strings or BigNumber objects to avoid precision errors

from drizzle-legacy.

For drizzle-box, I've narrowed down the issue to https://github.com/trufflesuite/drizzle-react-components/blob/master/src/AccountData.js#L36.

The issue with the code is that balance becomes undefined at some point during the account switch. I think a simple check for that should fix it if you have implemented your own version of AccountData. I'll open an issue + PR over on the drizzle-react-components repo.

I don't believe this is an issue with Drizzle itself, so I'll be closing this.

@terencechow Can you provide us with a sample of your code?

Or confirm that checking for undefined balance will fix your issue? If not, please let us know and we can look into this some more. Thanks.

from drizzle-legacy.

@honestbonsai can you bump and publish drizzle-react-components and then update the box so we can close this issue?

from drizzle-legacy.

Closing this out due to inactivity and @honestbonsai fixes in DRC and drizzle-box. Please reopen if necessary.

from drizzle-legacy.