You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As seen in #1268, the default currency manager is pretty heavy.
Originally, the RequestNetwork library knew about all currencies it could support, but since the introduction of CurrencyManager, it's highly configurable, and the expected usage is to specify the list of tokens.
Context
As seen in #1268, the default currency manager is pretty heavy.
Originally, the RequestNetwork library knew about all currencies it could support, but since the introduction of CurrencyManager, it's highly configurable, and the expected usage is to specify the list of tokens.
Proposal
Remove most currencies from the default currency list, especially Metamask's default list.
I suggest we keep:
Compatibility
For compatibility, we could offer an async method
loadExtendedCurrencyList
to fetch the Metamask list, as well as currently hard-coded tokens.The text was updated successfully, but these errors were encountered: