search

trustwallet / trust-wallet-ios

:iphone: Trust - Ethereum Wallet and Web3 DApp Browser for iOS
https://trustwallet.com
GNU General Public License v3.0
1.52k stars 721 forks source link

Remove CurrencyRate #846

Closed vikmeup closed 6 years ago

vikmeup commented 6 years ago

It is a dictionary that contains different prices, I think it could be removed and have the helper method in BalanceCoordinator to get a specific price since it has access to TokensDataStore

Current: price = rates[id] =>

balanceCoordinator.getPrice(for: Address)

OlegGordiichuk commented 6 years ago

After research I can conclude that we do not need BalanceCoordinator.

vikmeup commented 6 years ago

@OlegGordiichuk sounds like it fixed on master.