[bisq-network/bisq] Disambiguate payment acct json form's "country" field (#5330)
Stan
notifications at github.com
Thu Mar 18 20:24:39 CET 2021
Make it clear the user needs to enter a two letter country code in the api's payment account json form's `country` field, not a country name. A blank payment account json form now has a `country` default value = `your two letter country code`.
The json form's field name was not changed to countryCode for the following reason:
The API's payment account forms are dynamically generated using reflection, and `PaymentAccount` class hierarchy `@Setter` methods determine which form fields are included or excluded. Reflection and `@Setter` methods are also used when reading completed json forms, to set the field values on a new PaymentAccount instance before it is persisted.
`CountryBasedPaymentAccount` subclasses like `F2FAccount` have a `country` field and a `@Setter`, not a countryCode field, hence this shortcut to informing the user the country field value is a country-code.
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/5330
-- Commit Summary --
* Disambiguate payment acct json form's "country" field
-- File Changes --
M apitest/src/test/java/bisq/apitest/method/payment/AbstractPaymentAccountTest.java (5)
M core/src/main/java/bisq/core/api/model/PaymentAccountTypeAdapter.java (5)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/5330.patch
https://github.com/bisq-network/bisq/pull/5330.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/5330
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20210318/df141eba/attachment-0001.htm>
More information about the bisq-github
mailing list