[bisq-network/bisq] Add BTC address param, use String for param value instead of long (#1841)

Manfred Karrer notifications at github.com
Wed Oct 31 00:04:46 UTC 2018


ManfredKarrer commented on this pull request.



> @@ -59,120 +62,118 @@ public void validateDataFields(Proposal proposal) throws ValidationException {
         }
     }
 
-    // TODO
-    public void validateParamValue(Param param, long paramValue) throws ChangeParamValidationException {
-        // max 4 times the current value. min 25% of current value as general boundaries
-        checkMinMax(param, paramValue, 300, -75);
+    public void validateParamValue(Param param, String inputValue) throws ValidationException {

Yes we could add it the params

-- 
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/1841#discussion_r229528393
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20181030/d5856509/attachment.html>


More information about the bisq-github mailing list