Matt Luongo
Matt Luongo
@mr-michael calling your attention here: merging this PR means we'll pick up all / nearly all token balances at the risk of including scam tokens, though they'll have clear and...
@Shadowfiend @greg-nagy I opened https://github.com/tallycash/extension/issues/1893 to continue the asset trust discussion and unify the two approaches.
We don't rely soley on `alchemy_getAssetTransfers` now thankfully :) See #309 We do, however, still rely on `alchemy_getTokenBalances` in its absence. I don't know whether that has support on Kovan.
The plan is to fall back to manual balance checking per-token. It is indeed to a heavy client lift.
> Implementing getERC20TokenBalance (singular!) We have this already ☺️
Hmm, I thought we had resolved this cc @0xDaedalus
Pretty sure 1846 isn't the issue you were looking to reference @Shadowfiend
 The data looks right, but the page feels really crowded... not sure if that's the design. The NFT section not scrolling with the page makes it...
Here's what happens when I add `vitalik.eth` to the above 
> Isn't it expected? Vitalik.eth account is in the collapsed part (if you are asking where it is), you can toggle it with "view all". It is also very large...