[bisq-network/bisq] Fix bsq to fiat calculation (#4909)

chimp1984 notifications at github.com
Mon Dec 7 19:51:18 CET 2020


Fixes https://github.com/bisq-network/bisq/issues/4865

We used last trade price of BSQ for calculating the BSQ fee as fiat currency. This could distort the real amount in case the last trade had a price very different to average price.
We use now the 30 day average BSQ price. 
You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/4909

-- Commit Summary --

  * Move methods from AxisInlierUtils to InlierUtil
  * Remove empty class
  * Move methods to AveragePriceUtil
  * Use 30 day average BSQ price for calculating BSQ fee
  * Use preferredTradeCurrency is fiat, otherwise the users country currency

-- File Changes --

    M core/src/main/java/bisq/core/offer/OfferUtil.java (40)
    A core/src/main/java/bisq/core/util/AveragePriceUtil.java (139)
    A core/src/main/java/bisq/core/util/InlierUtil.java (140)
    M desktop/src/main/java/bisq/desktop/main/dao/economy/dashboard/BsqDashboardView.java (115)
    D desktop/src/main/java/bisq/desktop/main/dao/wallet/dashboard/BsqDashboardView.java (0)
    M desktop/src/main/java/bisq/desktop/util/AxisInlierUtils.java (116)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/4909.patch
https://github.com/bisq-network/bisq/pull/4909.diff

-- 
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/4909
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20201207/06bcf9e9/attachment.htm>


More information about the bisq-github mailing list