[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