From 9f3498a9dc94e86f5470f9c00fb9f5b3d626e0e1 Mon Sep 17 00:00:00 2001 From: Brian Bergeron Date: Thu, 6 Jun 2024 08:20:58 -0700 Subject: [PATCH] [TokenRatesController] - Keep addresses in checksum format (#4377) ## Explanation When adding market data for tokens in https://github.com/MetaMask/core/pull/4206, it also changed the token addresses to be lowercased instead of checksum. This PR moves it back to the checksum format, since some places on the client expected it. ## References ## Changelog ### `@metamask/assets-controllers` - **CHANGED**: Token rates controller uses checksum instead of lowercase format for token addresses ## Checklist - [ ] I've updated the test suite for new or updated code as appropriate - [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate - [ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate --- packages/assets-controllers/src/TokenRatesController.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/assets-controllers/src/TokenRatesController.ts b/packages/assets-controllers/src/TokenRatesController.ts index 0f0fa9cd327..71d4d8b7d24 100644 --- a/packages/assets-controllers/src/TokenRatesController.ts +++ b/packages/assets-controllers/src/TokenRatesController.ts @@ -555,7 +555,7 @@ export class TokenRatesController extends StaticIntervalPollingControllerV1< (obj, [tokenAddress, token]) => { obj = { ...obj, - [tokenAddress.toLowerCase()]: { ...token }, + [tokenAddress]: { ...token }, }; return obj;