Skip to content

Commit 1d2f649

Browse files
committed
fix: Nit refactor
1 parent ace5334 commit 1d2f649

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

packages/assets-controllers/CHANGELOG.md

+3-3
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
99

1010
### Added
1111

12-
- **BREAKING:** Add event `MultichainAssetsController:accountAssetListUpdated` in MultichainAssetsController to notify when new assets are detected for an account ([#5761](https://github.com/MetaMask/core/pull/5761))
12+
- Add event `MultichainAssetsController:accountAssetListUpdated` in MultichainAssetsController to notify when new assets are detected for an account ([#5761](https://github.com/MetaMask/core/pull/5761))
1313

1414
### Changed
1515

16-
- **BREAKING:** remove subscription to `MultichainAssetsController:stateChange` in `MultichainAssetsRatesController` and add subscription to `MultichainAssetsController:accountAssetListUpdated` ([#5761](https://github.com/MetaMask/core/pull/5761))
17-
- **BREAKING:** remove subscription to `MultichainAssetsController:stateChange` in `MultichainBalancesController` and add subscription to `MultichainAssetsController:accountAssetListUpdated` ([#5761](https://github.com/MetaMask/core/pull/5761))
16+
- **BREAKING:** Removed subscription to `MultichainAssetsController:stateChange` in `MultichainAssetsRatesController` and add subscription to `MultichainAssetsController:accountAssetListUpdated` ([#5761](https://github.com/MetaMask/core/pull/5761))
17+
- **BREAKING:** Removed subscription to `MultichainAssetsController:stateChange` in `MultichainBalancesController` and add subscription to `MultichainAssetsController:accountAssetListUpdated` ([#5761](https://github.com/MetaMask/core/pull/5761))
1818

1919
## [61.0.0]
2020

packages/assets-controllers/src/MultichainBalancesController/MultichainBalancesController.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -471,6 +471,7 @@ describe('MultichainBalancesController', () => {
471471
type: SolAccountType.DataAccount,
472472
},
473473
];
474+
474475
it('updates balances when receiving "MultichainAssetsController:accountAssetListUpdated" event and state is empty', async () => {
475476
const mockSolanaAccountId1 = mockListSolanaAccounts[0].id;
476477
const mockSolanaAccountId2 = mockListSolanaAccounts[1].id;

packages/assets-controllers/src/MultichainBalancesController/MultichainBalancesController.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ export class MultichainBalancesController extends BaseController<
186186
assets: [...added],
187187
}),
188188
);
189-
await this.#updateBalancesForAccounts(newAccountAssets);
189+
await this.#handleOnAccountAssetListUpdated(newAccountAssets);
190190
},
191191
);
192192
}
@@ -196,7 +196,7 @@ export class MultichainBalancesController extends BaseController<
196196
*
197197
* @param accounts - The accounts to update the balances for.
198198
*/
199-
async #updateBalancesForAccounts(
199+
async #handleOnAccountAssetListUpdated(
200200
accounts: {
201201
accountId: string;
202202
assets: CaipAssetType[];

0 commit comments

Comments
 (0)