[bisq-network/bisq] Add Tether token support via ERC20 and Omni (#4434)

chimp1984 notifications at github.com
Wed Aug 26 05:01:55 UTC 2020


@chimp1984 requested changes on this pull request.



> @@ -535,6 +535,10 @@ private String getPriceProviderTooltipString() {
         String selectedCurrencyCode = model.getPriceFeedService().getCurrencyCode();
         MarketPrice selectedMarketPrice = model.getPriceFeedService().getMarketPrice(selectedCurrencyCode);
 
+        if (selectedMarketPrice == null) {

I think that is better:

```
private String getPriceProviderTooltipString() {
        String selectedCurrencyCode = model.getPriceFeedService().getCurrencyCode();
        MarketPrice selectedMarketPrice = model.getPriceFeedService().getMarketPrice(selectedCurrencyCode);
        return Res.get("mainView.marketPrice.tooltip",
                "Bisq Price Index for " + selectedCurrencyCode,
                "",
                selectedMarketPrice != null ? DisplayUtils.formatTime(new Date(selectedMarketPrice.getTimestampSec())) : Res.get("shared.na"),
                model.getPriceFeedService().getProviderNodeAddress());
    }```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4434#pullrequestreview-475151352
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200825/1f5db9ab/attachment.html>


More information about the bisq-github mailing list