Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NEP-5 balances not displaying for some accounts #2575

Closed
hal0x2328 opened this issue Nov 13, 2023 · 3 comments
Closed

NEP-5 balances not displaying for some accounts #2575

hal0x2328 opened this issue Nov 13, 2023 · 3 comments
Labels

Comments

@hal0x2328
Copy link
Member

hal0x2328 commented Nov 13, 2023

Legacy NEP-5 balance display is broken under some conditions - one condition is if the user has ever provided liquidity to a legacy Flamingo pool.

The issue is related to CityOfZion/neon-js#921 - when the token info call fails due to neon-js' inability to decode the totalSupply (even on a LP pair token with no remaining balance), Neon no longer continues trying to retrieve token balances for the rest of the NEP-5 tokens in the wallet.

If possible, Neon should try and handle the token info error gracefully and just skip any it can't decode (I don't believe totalSupply is even used anywhere in Neon)

Here's an example for reproduction:

  1. Import a watch account for the address AKEX765mtnPUPj4L5n5C8KcauGkRkuTuxQ
  2. Neon shows the following error in the debug tools console:
bundle.js:formatted:525160 Error: Expected a hexstring but got 33119936322
    at S (bundle.js:formatted:39878:27)
    at M (bundle.js:formatted:39863:17)
    at Array.O (bundle.js:formatted:69284:57)
    at bundle.js:formatted:42405:36
    at Array.map (<anonymous>)
    at f.parse (bundle.js:formatted:42404:30)
    at bundle.js:formatted:41279:27
  1. No NEP-5 token balances are displayed, despite this address having a balance of nNEO and fUSDT
Copy link

This issue is stale because it has been open for 14 days with no activity. It will be automatically closed in 7 days from now if no activity is detected.

@github-actions github-actions bot added the stale label Jan 13, 2024
Copy link

This issue was closed because it has been inactive for 7 days since being marked as stale.

@jseagrave21
Copy link
Contributor

@hal0x2328 I started working on this problem and was able to resolve the issue with Legacy nep-5 transactions not showing up on the Activity tab. However, balances are still not correctly displayed and I think that is because the master token list has not been up-to-date. I checked the address you listed and NeoTube shows a balance of fUSDT at the time of writing this, which is not included in the master list of tokens. I am not sure who is maintaining that list anymore but it looks like @comountainclimber was the last person to merge an update to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants