[bisq-network/bisq] Add fiat equivalent for Total Amount & Security Deposit for both makers and sellers (#5452)
BtcContributor
notifications at github.com
Fri Apr 30 18:33:59 CEST 2021
Now the recap of the trade displays also the fiat equivalent for Total Amount & Security Deposit for both makers and sellers.
Fix also the Total Amount displayed as a taker of a Sell btc trade when using bsq as trading fees (Bsq was not displayed in Total Amount).
**Maker, btc trading fee, previously**
<img width="452" alt="maker_btc_old" src="https://user-images.githubusercontent.com/79100296/116713854-791cf080-a9d5-11eb-8ea6-d6b8c6d97bae.png">
**Maker, btc trading fee, now**
<img width="449" alt="maker_btc_new" src="https://user-images.githubusercontent.com/79100296/116720169-2d217a00-a9dc-11eb-8d1d-f827f4a51bde.png">
**Taker, btc trading fee, previously**
<img width="458" alt="taker_btc_old" src="https://user-images.githubusercontent.com/79100296/116725147-dcad1b00-a9e1-11eb-9764-2d879a7829ac.png">
**Taker, btc trading fee, now**
<img width="457" alt="taker_btc_new" src="https://user-images.githubusercontent.com/79100296/116720192-314d9780-a9dc-11eb-9ca7-0a288800464d.png">
**Maker, bsq trading fee, previously**
<img width="444" alt="maker_bsq_old" src="https://user-images.githubusercontent.com/79100296/116713988-9356ce80-a9d5-11eb-8612-99099138f6d7.png">
**Maker, bsq trading fee, now**
<img width="446" alt="maker_bsq_new" src="https://user-images.githubusercontent.com/79100296/116720208-36124b80-a9dc-11eb-90df-cc7fd08c74b8.png">
**Taker, bsq trading fee, previously** -> THIS IS INCORRECT AS BSQ TRADING FEES ARE NOT DISPLAYED IN TOTAL AMOUNT
<img width="447" alt="taker_bsq_old" src="https://user-images.githubusercontent.com/79100296/116723438-d61da400-a9df-11eb-9990-5b3a8f92d2cb.png">
**Taker, bsq trading fee, now**
<img width="448" alt="taker_bsq_new" src="https://user-images.githubusercontent.com/79100296/116720222-3a3e6900-a9dc-11eb-832f-6d8048d92577.png">
When security deposit is not the minimum for example, this is how it looks like:
<img width="448" alt="percentage" src="https://user-images.githubusercontent.com/79100296/116721525-aff70480-a9dd-11eb-89c1-4bcd8bc20668.png">
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/5452
-- Commit Summary --
* Add fiat equivalent for Total Amount & Security Deposit for both makers and sellers
* Restore totalToPay as 'final' as before
* Notice when the security deposit is the minimum possible
* Add fiat conversion also for trade amount when making a trade to sell btc
-- File Changes --
M desktop/src/main/java/bisq/desktop/main/offer/MutableOfferViewModel.java (33)
M desktop/src/main/java/bisq/desktop/main/offer/OfferDataModel.java (2)
M desktop/src/main/java/bisq/desktop/main/offer/takeoffer/TakeOfferView.java (2)
M desktop/src/main/java/bisq/desktop/main/offer/takeoffer/TakeOfferViewModel.java (33)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/5452.patch
https://github.com/bisq-network/bisq/pull/5452.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/5452
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20210430/e590c984/attachment.htm>
More information about the bisq-github
mailing list