search

OCA / l10n-finland

GNU Affero General Public License v3.0
4 stars 20 forks source link

[10.0][ADD] business code and validation #16

Closed jarmokortetjarvi closed 6 years ago

jarmokortetjarvi commented 6 years ago

Adds a business id for partners. Split to two modules: one just adds the field and other adds validation and formatting. This way the validation will be optional, and the module can be installed in environments with incorrectly formatted business ids

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-1.7%) to 98.333% when pulling 51fa60e9d20891c85229892fcdf34221f8c80130 on Tawasta:10.0-l10n_fi_business_code into 5aee1230995891cfd13ee76f6afc11fe0eb87273 on OCA:10.0.

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 1240cca76a2a3a1ca66549607f45e769fe11d6b0 on Tawasta:10.0-l10n_fi_business_code into 19b1749632c04f0e97a7359b6782435f7ad696ed on OCA:10.0.

jarmokortetjarvi commented 6 years ago

Tried to clean up the commit history. If everything went as planned, it should be identical to the original PR (with fixes of course), but without any ./setup-files.

Pre-cleanup branch for reference https://github.com/Tawasta/l10n-finland/tree/10.0-l10n_fi_business_code_original