[bisq-network/bisq] Update trigger price validation (PR #6109)

xyzmaker123 notifications at github.com
Mon Mar 21 11:56:46 CET 2022


> Did you also check the trigger price actual execution code? I'd surprised, if only changing the UI validation will do the work as expected.

@ripcurlx I roughly reviewed [it](https://github.com/bisq-network/bisq/blob/f7fcaa915e0467ce51a41688685ae6d557b39443/core/src/main/java/bisq/core/offer/bisq_v1/TriggerPriceService.java#L116-L185). I think changing validation criteria is enough - we basically add more flexibility to user to set trigger price possible values, but execution logic doesn't change.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/6109#issuecomment-1073753882
You are receiving this because you are subscribed to this thread.

Message ID: <bisq-network/bisq/pull/6109/c1073753882 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20220321/2a9e9b82/attachment.htm>


More information about the bisq-github mailing list