[16.0][FIX] l10n_it_riba: add migrations to update it from 14.0

[odooNextev]

Questa PR serve per rinominare il modulo da l10n_it_ricevute_bancarie a l10n_it_riba nella migrazione alla versione 16.0.

[odooNextev]

OCA/OpenUpgrade#4218

[odooNextev]

@SirAionTech ho aggiunto questo commit per risolvere il TODO che avevi lasciato: 878049a
Pensi che vada bene?

[SirAionTech]

Grazie della PR!
Giusto per info, stai provando la migrazione con Odoo SA giusto?

[odooNextev]

Grazie della PR! Giusto per info, stai provando la migrazione con Odoo SA giusto?

Sì

[SirAionTech]

Si potrebbe schiacciare tutto in un commit o c'è un motivo particolare per cui li vuoi far mergiare come separati?

[odooNextev]

Si potrebbe schiacciare tutto in un commit

Fatto

[SirAionTech]

I test sono falliti per #3801, vediamo se ora vanno.
/ocabot rebase

[OCA-git-bot]

Congratulations, PR rebased to 16.0.

[SirAionTech]

Grazie mille delle mille modifiche!
Ho provato la migrazione con OpenUpgrade e per me è ok: la RiBa viene creata con le sue righe e lo stato corretto.

Per simulare la migrazione con Enterprise ho rimosso https://github.com/OCA/OpenUpgrade/blob/d011acded3efb72550befd8e212ea8a88a6ea093/openupgrade_scripts/apriori.py#L52 ed eseguito la migrazione con OpenUpgrade.
Nel DB migrato c'erano giustamente sia il modulo vecchio che quello nuovo; ho installato il nuovo ed ha assorbito quello vecchio, e i dati della RiBa erano di nuovo corretti.

Ho guardato il codice ed è coerente con https://github.com/OCA/l10n-italy/wiki/Migrazione-con-rinomina-modulo,-compatibile-con-OpenUpgrade-e-Odoo-SA.

Non è bloccante ma ho notato solo ora che c'è ancora

l10n-italy/l10n_it_riba/readme/ROADMAP.md

Lines 3 to 4 in dc96463

	Attenzione! Il modulo è stato migrato, ma gli script di migrazione non
	sono stati provati.

lo potresti rimuovere? Direi che entrambi abbiamo provato la migrazione sufficientemente.
Poi per me si può mergiare.

[odooNextev]

@SirAionTech fatto

[SirAionTech]

@SirAionTech fatto

Avevo appena fatto rebase e c'era solo da rimuovere un file, però il diff delle ultime modifiche https://github.com/OCA/l10n-italy/compare/8c584d3901419f43f383d3c64e2d43bc2c9cdc43..c4199ef1b9dd5295208976a5804c489bf5d15ac5 riporta più di 40 file modificati.
Mi sembra sia perché il tuo branch https://github.com/odooNextev/l10n-italy/tree/16.0-fix-l10n_it_riba-mig applica il commit di questa PR a una versione vecchia di 16.0.
Faccio /ocabot rebase per cercare di isolare le ultime modifiche.

[OCA-git-bot]

Hi @SirAionTech. Your command failed:

Invalid options for command rebase: per cercare di isolare le ultime modifiche.

Ocabot commands

• ocabot merge major|minor|patch|nobump
• ocabot rebase
• ocabot migration {MODULE_NAME}

More information

• ocabot documentation
• OCA guidelines, specially the "Version Numbers" section.

[SirAionTech]

Hi @SirAionTech. Your command failed:

Invalid options for command rebase: per cercare di isolare le ultime modifiche.

Ocabot commands

• ocabot merge major|minor|patch|nobump
• ocabot rebase
• ocabot migration {MODULE_NAME}

More information

• ocabot documentation
• OCA guidelines, specially the "Version Numbers" section.

Sorry
/ocabot rebase

[OCA-git-bot]

Congratulations, PR rebased to 16.0.

[SirAionTech]

Ok https://github.com/OCA/l10n-italy/compare/8c584d3901419f43f383d3c64e2d43bc2c9cdc43..e4565fed3a160572f336f1048929387e3dbf408b ha più senso: include le modifiche di #4434 che è stata mergiata nel frattempo, ma per il resto c'è solo il file rimosso 👍

/ocabot merge nobump

[odooNextev]

@SirAionTech fatto

Avevo appena fatto rebase e c'era solo da rimuovere un file, però il diff delle ultime modifiche https://github.com/OCA/l10n-italy/compare/8c584d3901419f43f383d3c64e2d43bc2c9cdc43..c4199ef1b9dd5295208976a5804c489bf5d15ac5 riporta più di 40 file modificati. Mi sembra sia perché il tuo branch https://github.com/odooNextev/l10n-italy/tree/16.0-fix-l10n_it_riba-mig applica il commit di questa PR a una versione vecchia di 16.0. Faccio /ocabot rebase per cercare di isolare le ultime modifiche.

Mi ero scordato di aggiornare il mio branch locale...

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-3703-by-SirAionTech-bump-nobump, awaiting test results.

[OCA-git-bot]

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