MetaMask / MetaMask/core

TokenRatesController: Refactor

Open
#1,612 0 comments 0 reactions 0 assignees View on GitHub
team-assets team-wallet-framework wf-controller-tech-debt
Dominant language
TypeScript
Stars
413
Forks
308
Avg merge
1d 4h
Merged PRs (30d)
253

Description

The `fetchAndMapExchangeRates` method seems to be complex. We should split this up, extracting logic to separate files, so that it is easier to understand and test.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the TokenRatesController and its fetchAndMapExchangeRates method; no file or test path is specified in the issue. Trace the method's current responsibilities and existing coverage before deciding how to separate them, then verify that behavior remains unchanged and the resulting logic is easier to test.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend-api-design
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.