OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
142 stars 297 forks source link

[14.0][FIX] l10n_it_fiscalcode: add codicefiscale.isvalid() and check when partner becomes a person #4180

Closed toita86 closed 3 weeks ago

toita86 commented 1 month ago

Backporting di

sergiocorato commented 1 month ago

Solo un appunto: il changelog riporta i riferimenti alla 16.0, qui dovrebbe essere allineato alla 14.0 direi.

francesco-ooops commented 1 month ago

@sergiocorato io lo rimuoverei, non credo ci sia bisogno di un file history per questo modulo

francesco-ooops commented 1 month ago

@sergiocorato go?

primes2h commented 1 month ago

@toita86 Questa PR fa il backport anche del FIX di questa issue: https://github.com/OCA/l10n-italy/issues/3719 (vedi commit https://github.com/OCA/l10n-italy/pull/4180/commits/5b5b4b632945349227ec8e85b38b4c1369c59a66)

Puoi aggiornare la descrizione della PR e aggiungere la PR stessa alla issue?

francesco-ooops commented 4 weeks ago

@primes2h va bene ora?

sergiocorato commented 4 weeks ago

@primes2h va bene ora?

Nel primo commit c'è ancora il file HISTORY.rst, rimosso dal secondo

primes2h commented 3 weeks ago

/ocabot merge patch

OCA-git-bot commented 3 weeks ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-4180-by-primes2h-bump-patch, awaiting test results.

OCA-git-bot commented 3 weeks ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot commented 3 weeks ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot commented 3 weeks ago

It looks like something changed on 14.0 in the meantime. Let me try again (no action is required from you). Prepared branch 14.0-ocabot-merge-pr-4180-by-primes2h-bump-patch, awaiting test results.

OCA-git-bot commented 3 weeks ago

It looks like something changed on 14.0 in the meantime. Let me try again (no action is required from you). Prepared branch 14.0-ocabot-merge-pr-4180-by-primes2h-bump-patch, awaiting test results.

OCA-git-bot commented 3 weeks ago

Congratulations, your PR was merged at 78db1373aff73ec1a0285f6976f88a59c42fd0b9. Thanks a lot for contributing to OCA. ❤️