MetaMask / MetaMask/core

Merge Extension AccountsTrackerController with the core one

Open
#1,809 1 comment 0 reactions 1 assignee Assigned to @salimtb View on GitHub
team-assets
Dominant language
TypeScript
Stars
413
Forks
308
Avg merge
1d 4h
Merged PRs (30d)
253

Description

These controllers are similar but have separate implementations. We will need to dive into how they are used in the extension vs. mobile app to understand what a combined version should look like.

While we are doing this, we may need to:

- Extract any API interactions or complex utility functions
- Refactor to be more easily understood
- Write comprehensive unit tests

Additionally, there are patches in mobile around AccountTrackerController under `@metamask/assets-controllers`, and we will want to make sure those are integrated as well.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.