[bisq-network/bisq] Fix two CLI option parsing bugs (#5063)
Stan
notifications at github.com
Fri Jan 8 15:17:25 CET 2021
- Fix bug locating index of method name in String[] args when handling any negative number opt.
- Make `createoffer` method's security-deposit param format consistent with UI's.
When creating an offer, the CLI should take "15.0", not "0.15" as a 15% security deposit. This is consistent with the UI, and the CLI's mkt-price-margin input format.
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/5063
-- Commit Summary --
* Fix two CLI option parsing bugs
-- File Changes --
M cli/src/main/java/bisq/cli/CliMain.java (5)
M cli/src/main/java/bisq/cli/CurrencyFormat.java (11)
M cli/src/main/java/bisq/cli/NegativeNumberOptions.java (22)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/5063.patch
https://github.com/bisq-network/bisq/pull/5063.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/5063
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20210108/6159b122/attachment.htm>
More information about the bisq-github
mailing list