Description

The AccountsApiBalanceFetcher had a bug caused by address format incompatibility between controllers:

• TokenBalancesController uses lowercase addresses for ERC20 tokens
• AccountTrackerController uses checksum addresses for native tokens

The original code always checksummed all addresses, causing a wrong token balances state

core PR: MetaMask/core#6812

Changelog

CHANGELOG entry: fix balances issue update when api fetcher is used

Related issues

Fixes: #21030

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.