Skip to content

[18.0] [MIG] l10n_it_riba_oca#4649

Merged
OCA-git-bot merged 50 commits into
OCA:18.0from
odooNextev:18.0-mig-l10n_it_riba_oca
Nov 6, 2025
Merged

[18.0] [MIG] l10n_it_riba_oca#4649
OCA-git-bot merged 50 commits into
OCA:18.0from
odooNextev:18.0-mig-l10n_it_riba_oca

Conversation

@odooNextev

@odooNextev odooNextev commented Mar 17, 2025

Copy link
Copy Markdown
Contributor

Ho lasciato diversi commenti nel codice perchè i campi codice fiscale, CIG e CUP nella 18.0 sono aggiunti dal modulo l10n_it_edi.
Dobbiamo decidere se far dipendere il modulo dalla fatturazione elettronica anche se le Ri.Ba potrebbero essere usate anche senza.
Effettivamente anche nelle vecchie versioni questo modulo dipendeva da l10n_it_fatturapa_out, ma potremmo fare un modulo ponte con l10n_it_edi.

Avrei anche voluto mettere il modulo l10n_it_riba di Odoo SA negli excludes, ma inizialmente il modulo omonimo di OCA deve essere installato.

Depends on:

@primes2h

Copy link
Copy Markdown
Contributor

/ocabot migration l10n_it_riba

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Mar 17, 2025
@stenext stenext force-pushed the 18.0-mig-l10n_it_riba_oca branch from db4340b to c9a1a93 Compare March 17, 2025 13:25
@odooNextev

Copy link
Copy Markdown
Contributor Author

@eLBati @MarcoCalcagni @TheMule71 @primes2h @MaurizioPellegrinet @salvorapi quando riuscite avrei bisogno di revisioni.
Oltre all'aspetto funzionale, praticamente è uguale a quello che c'è ora sulla 16.0, ci sono dei punti da discutere nello sviluppo che ho messo nella descrizione della PR.
Ci sarebbe da revisionare anche la PR di @Borruso #4505 per la migrazione del modulo l10n_it_abicab dalla quale dipende quello delle Ri.Ba, a me sembra funzionare anche quello.

@primes2h

Copy link
Copy Markdown
Contributor

Ho lasciato diversi commenti nel codice perchè i campi codice fiscale, CIG e CUP nella 18.0 sono aggiunti dal modulo l10n_it_edi. Dobbiamo decidere se far dipendere il modulo dalla fatturazione elettronica anche se le Ri.Ba potrebbero essere usate anche senza. Effettivamente anche nelle vecchie versioni questo modulo dipendeva da l10n_it_fatturapa_out, ma potremmo fare un modulo ponte con l10n_it_edi.

Come indichi giustamente tu il modulo l10n_it_edi non fornisce solo la funzionalità base della fatturazione elettronica ma definisce anche tutta una serie di campi fondamentali per altre operazioni.
Quindi non ci vedrei nulla di male a far dipendere l10n_it_riba_oca direttamente da l10n_it_edi.

Avrei anche voluto mettere il modulo l10n_it_riba di Odoo SA negli excludes, ma inizialmente il modulo omonimo di OCA deve essere installato.

Non ho capito cosa intendi.

@odooNextev

Copy link
Copy Markdown
Contributor Author

@primes2h ok, grazie del parere.
Appena riesco aggiungo la dipendenza e decommento le parti che avevo lasciato da parte.

Parlavo di mettere il modulo l10n_it_riba di Odoo SA negli excludes per evitare che vengano installati entrambi per sbaglio, ma se un utente migra dalla 16 ha bisogno che il modulo l10n_it_riba (di OCA) sia installato e quindi ci sarebbe un conflitto.

@primes2h

Copy link
Copy Markdown
Contributor

@primes2h ok, grazie del parere. Appena riesco aggiungo la dipendenza e decommento le parti che avevo lasciato da parte.

Parlavo di mettere il modulo l10n_it_riba di Odoo SA negli excludes per evitare che vengano installati entrambi per sbaglio, ma se un utente migra dalla 16 ha bisogno che il modulo l10n_it_riba (di OCA) sia installato e quindi ci sarebbe un conflitto.

IIRC la rinomina del modulo dovebbe avvenire prima dell'effettivo aggiornamento del modulo stesso quindi in teoria il problema non sussiste.

https://github.com/OCA/l10n-italy/wiki/Migrazione-con-rinomina-modulo,-compatibile-con-OpenUpgrade-e-Odoo-SA

@odooNextev

Copy link
Copy Markdown
Contributor Author

IIRC la rinomina del modulo dovebbe avvenire prima dell'effettivo aggiornamento del modulo stesso quindi in teoria il problema non sussiste.

https://github.com/OCA/l10n-italy/wiki/Migrazione-con-rinomina-modulo,-compatibile-con-OpenUpgrade-e-Odoo-SA

Ottimo, grazie.

@primes2h

Copy link
Copy Markdown
Contributor

@primes2h ok, grazie del parere. Appena riesco aggiungo la dipendenza e decommento le parti che avevo lasciato da parte.
Parlavo di mettere il modulo l10n_it_riba di Odoo SA negli excludes per evitare che vengano installati entrambi per sbaglio, ma se un utente migra dalla 16 ha bisogno che il modulo l10n_it_riba (di OCA) sia installato e quindi ci sarebbe un conflitto.

IIRC la rinomina del modulo dovebbe avvenire prima dell'effettivo aggiornamento del modulo stesso quindi in teoria il problema non sussiste.

https://github.com/OCA/l10n-italy/wiki/Migrazione-con-rinomina-modulo,-compatibile-con-OpenUpgrade-e-Odoo-SA

Confermo:
https://github.com/OCA/OpenUpgrade/blob/edd469e3979adae28ad87b18f7d8d3fbe44fdfd9/openupgrade_scripts/scripts/base/16.0.1.3/pre-migration.py#L162

@eLBati

eLBati commented Mar 21, 2025

Copy link
Copy Markdown
Member

/ocabot migration l10n_it_riba_oca

@stenext stenext force-pushed the 18.0-mig-l10n_it_riba_oca branch 2 times, most recently from c04c0f8 to dcc3de5 Compare March 26, 2025 13:29
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Apr 18, 2025
@stenext stenext force-pushed the 18.0-mig-l10n_it_riba_oca branch 8 times, most recently from 6fd68a0 to 2d324f4 Compare June 6, 2025 16:04
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Jun 13, 2025
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Jul 14, 2025
SirAionTech and others added 16 commits October 27, 2025 15:15
Also allow to set a date during payment and past due process
Currently translated at 100.0% (244 of 244 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_riba
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_riba/it/
Currently translated at 100.0% (248 of 248 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_riba
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_riba/it/
@stenext stenext force-pushed the 18.0-mig-l10n_it_riba_oca branch 2 times, most recently from 102a8f7 to f345af4 Compare October 27, 2025 14:57
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Oct 31, 2025

@eLBati eLBati left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eLBati

eLBati commented Nov 6, 2025

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-4649-by-eLBati-bump-nobump, awaiting test results.

@Borruso

Borruso commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

cig = fields.Char(string="CIG", size=256)
cup = fields.Char(string="CUP", size=256)

il cig e cup sono ancora char non avevamo detto che dovevano essere compute?

farò PR dopo

@odooNextev

Copy link
Copy Markdown
Contributor Author

cig = fields.Char(string="CIG", size=256) cup = fields.Char(string="CUP", size=256)

il cig e cup sono ancora char non avevamo detto che dovevano essere compute?

lo sto facendo adesso, non ho visto che c'era in corso il merge, faccio una pr di correzione tra un attimo

@OCA-git-bot OCA-git-bot merged commit 7f49e0c into OCA:18.0 Nov 6, 2025
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.