Fix: Prevented incorrect error handling on getting token balances and corrected cGHS address for Alfajores

[Andrew718PLTS]

Description

We couldn't get balances on Alfajores because of the incorrect error handling on getting token balances - it was trying to get the balance of the token with the address that is not correct (cGHS, this way we faced the issue). Now, I replaced it with try catch block to prevent throwing an error when there's an issue with some of the tokens. Instead, we load all the tokens we can and display the responsive console error to make it more understandable and prevent problems like this.

Other changes

Updated address for the 'cGHS' token.

Tested

While token/s addresses are correct.

1. Navigate to the preview and connect your wallet where you have tokens on the Alfajores.
2. Change the network to Alfajores.
3. Wait until the balances are loaded.

While token/s addresses are not correct

1. Check out the branch and change any token address to an incorrect one.
2. Run locally and connect your wallet where you have tokens on the Alfajores.
3. Change the network to Alfajores.
4. Wait until the warning notification toast is loaded and the console error is displayed.

Related issues

• Fixes #issue number here

Checklist before requesting a review

• I have performed a self-review of my own code
• I have added tests that prove my fix is effective or that my feature works
• The PR title follows the conventions
• I have run the regression tests

[linear]

SUP-166 [Mento-Web] Can't load balances for the Alfajores testnet

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
mento-web	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 11, 2025 10:04am

[Copilot]

PR Overview

This PR fixes the error handling issue when fetching token balances on Alfajores and updates the cGHS token address. The changes include:

• Refactoring the token balance fetching logic to use a try/catch block to prevent the entire operation from failing for a single token.
• Updating the error messaging in the polling worker.
• Correcting the cGHS token address in the token configuration.

Reviewed Changes

File	Description
src/features/accounts/fetchBalances.ts	Refactor to use getTokenBalance with try/catch and update balance fetching.
src/features/polling/PollingWorker.tsx	Update error message and logging for balance fetching failures.
src/config/tokens.ts	Update the cGHS token address for Alfajores.

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/features/accounts/fetchBalances.ts:41

• getTokenBalance returns undefined when an error occurs (see lines 54-57), but tokenBalances is force-cast to a mapping of string values, which may lead to a type inconsistency. Consider handling undefined balance values or updating the return type accordingly.

return tokenBalances as Record<TokenId, string>

[chapati23]

tested this on the preview url on alfajores and it fetched balances correctly

nice work on improving the error handling for balance fetching as well