MetaMask / MetaMask/core

TokenRatesController Technical Debt

Open
#4,404 0 comments 0 reactions 1 assignee Claimed by @salimtb 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 TokenRatesController state which is mapping of token address and exchange rate used to be managed with `ContractExchangeRates` type. But recently, it's completely modified and new `ContractMarketData` has been added to manage it's state. Now there's no need of `ContractExchangeRates` and needs to be removed.

This ticket, requires to analyse if there is still ContractExchangeRates used where the ContractMarketData build included. And remove all it's occurrences and also rename the variables and functions which are still pre-fixed / post-fixed with ContractExchangeRate.

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.