search

OCA / l10n-italy

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

[FIX] l10n_it_fiscalcode - Cambiando tipologia del partner non viene eseguito il controllo del CF #4076

Closed SirAionTech closed 5 months ago

SirAionTech commented 5 months ago

Risolve https://github.com/OCA/l10n-italy/issues/4050 per 16.0.

SirAionTech commented 5 months ago

@odooNextev Dal test su runboat ho notato un comportamento anomalo. Se parto da un partner di tipo azienda che contiene un CF non corretto e cambio la tipologia in persona fisica, al salvataggio non viene eseguito alcun controllo. Ho visto però che il problema era presente anche prima di questa PR, quindi possiamo comunque procedere con il merge. È giusto però segnalare la cosa, casomai qualcuno volesse darci un'occhiata.

Puoi aprire una issue (https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo#apertura-issue)? Sarà comunque difficile che qualcuno si metta a correggerlo, ma almeno ha più visibilità di un commento in una PR

Fatto https://github.com/OCA/l10n-italy/issues/4050

Originally posted by @primes2h in https://github.com/OCA/l10n-italy/issues/3724#issuecomment-1994717539

Come non detto :smile: puoi verificare se corregge la issue https://github.com/OCA/l10n-italy/issues/4050 che avevi aperto?

primes2h commented 5 months ago

@odooNextev Dal test su runboat ho notato un comportamento anomalo. Se parto da un partner di tipo azienda che contiene un CF non corretto e cambio la tipologia in persona fisica, al salvataggio non viene eseguito alcun controllo. Ho visto però che il problema era presente anche prima di questa PR, quindi possiamo comunque procedere con il merge. È giusto però segnalare la cosa, casomai qualcuno volesse darci un'occhiata.

Puoi aprire una issue (https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo#apertura-issue)? Sarà comunque difficile che qualcuno si metta a correggerlo, ma almeno ha più visibilità di un commento in una PR

Fatto #4050

Originally posted by @primes2h in #3724 (comment)

Come non detto 😄 puoi verificare se corregge la issue #4050 che avevi aperto?

:smile: Direi di si. :+1:

primes2h commented 5 months ago

/ocabot merge patch

OCA-git-bot commented 5 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-4076-by-primes2h-bump-patch, awaiting test results.

OCA-git-bot commented 5 months ago

@primes2h The merge process could not be finalized, because command twine upload --repository-url https://upload.pypi.org/legacy/ -u __token__ odoo_addon_l10n_it_fiscalcode-16.0.1.0.2-py3-none-any.whl failed with output:

Uploading distributions to https://upload.pypi.org/legacy/
Uploading odoo_addon_l10n_it_fiscalcode-16.0.1.0.2-py3-none-any.whl
[?25l
  0% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 0.0/374.9 kB • --:-- • ?
  0% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 0.0/374.9 kB • --:-- • ?
  0% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 0.0/374.9 kB • --:-- • ?
 22% ━━━━━━━━╸━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 81.9/374.9 kB • 00:01 • 74.7 MB/s
 46% ━━━━━━━━━━━━━━━━━━╺━━━━━━━━━━━━━━━━━━━━━ 172.0/374.9 kB • 00:01 • 1.5 MB/s
 74% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╸━━━━━━━━━━ 278.5/374.9 kB • 00:01 • 1.2 MB/s
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 374.9/374.9 kB • 00:00 • 1.1 MB/s
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 374.9/374.9 kB • 00:00 • 1.1 MB/s
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 374.9/374.9 kB • 00:00 • 1.1 MB/s
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 374.9/374.9 kB • 00:00 • 1.1 MB/s
[?25hWARNING  Error during upload. Retry with the --verbose option for more details. 
ERROR    HTTPError: 403 Forbidden from https://upload.pypi.org/legacy/          
         New uploads are temporarily disabled. See                              
         https://pypi.org/help/#admin-intervention for more information.