[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