[bisq-network/bisq] Validate phone numbers (#3149)

Stan notifications at github.com
Sat Aug 31 18:36:14 UTC 2019


@ripcurlx After you've tested this PR, and after (if) you merge it, I have HalCash phone validation ready for a new branch & PR.  (FYI, HalCash required a couple of changes to pb.proto. So may others.) 

What do you think of grouping phone validation PRs into similar payment methods?  

For example, the next PR will be "Validate HalCash phone number", then a few to several (depending on similarity of changes needed) phone-validator integrations can be grouped into the next PR, and so on.  

Each PR would benefit users of each affected payment method, and testers wouldn't need to test everything all at once. 

-- 
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/3149#issuecomment-526855474
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190831/a0088f74/attachment.html>


More information about the bisq-github mailing list