Skip to content

[18.0] [MIG] l10n_it_account: Migration to 18.0#4555

Merged
OCA-git-bot merged 90 commits into
OCA:18.0from
Borruso:18.0-mig-l10n_it_account
Apr 10, 2025
Merged

[18.0] [MIG] l10n_it_account: Migration to 18.0#4555
OCA-git-bot merged 90 commits into
OCA:18.0from
Borruso:18.0-mig-l10n_it_account

Conversation

@Borruso

@Borruso Borruso commented Jan 17, 2025

Copy link
Copy Markdown
Contributor

Issue: #4392

@Borruso Borruso force-pushed the 18.0-mig-l10n_it_account branch 8 times, most recently from 2da7954 to 583e954 Compare January 17, 2025 15:50
Comment thread l10n_it_account/__init__.py Outdated
openupgrade.rename_fields(
env,
field_spec,
)

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.

Visto che l10n_it_account esiste anche nella 16, questo script sarebbe da mettere in migrations

@eLBati eLBati mentioned this pull request Jan 17, 2025
46 tasks
@Borruso Borruso force-pushed the 18.0-mig-l10n_it_account branch from 583e954 to 9a2b3a4 Compare January 24, 2025 09:26
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Jan 24, 2025
@Borruso Borruso force-pushed the 18.0-mig-l10n_it_account branch 2 times, most recently from b3b83d4 to 25410d5 Compare January 24, 2025 16:11
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Jan 24, 2025
@Borruso Borruso force-pushed the 18.0-mig-l10n_it_account branch from 25410d5 to ae6cd35 Compare January 31, 2025 08:27
Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Jan 31, 2025

@primes2h primes2h left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nel commit di migrazione è stato tolto un bel po' di codice presente nella v.16.
Puoi controllare ?

f869c55

@Borruso Borruso force-pushed the 18.0-mig-l10n_it_account branch from ae6cd35 to 35ea546 Compare February 4, 2025 17:00
@eLBati

eLBati commented Feb 6, 2025

Copy link
Copy Markdown
Member

Nel commit di migrazione è stato tolto un bel po' di codice presente nella v.16. Puoi controllare ?

f869c55

https://www.odoo-italia.org/web#id=727&cids=1&menu_id=206&action=328&active_id=18&model=project.task&view_type=form qua potrebbe esserci qualche risposta

@MarcoCalcagni

Copy link
Copy Markdown

da aggiungere in migrazione pec_mail -> l10n_it_pec_email

@primes2h

Copy link
Copy Markdown
Contributor

Nel commit di migrazione è stato tolto un bel po' di codice presente nella v.16. Puoi controllare ?
f869c55

https://www.odoo-italia.org/web#id=727&cids=1&menu_id=206&action=328&active_id=18&model=project.task&view_type=form qua potrebbe esserci qualche risposta

Ah, grazie mille!
Però è stato tolto anche il codice relativo a questo commit, che andrebbe lasciato.
3ef0e3b

"specifies the VAT register (sales / purchases) where the "
"tax must be computed.",
)
parent_tax_ids = fields.Many2many(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Borruso ma già sulla 18.0 c'è il campo "children_tax_ids"

https://github.com/odoo/odoo/blob/ac9f4e173e4a2fbaaad1cbad34bd0f26dc3a22a4/addons/account/models/account_tax.py#L118

E' necessario ripeterlo in questo caso ?

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.

children_tax_ids

@salvorapi children_tax_ids sono i figli , parent_tax_ids i genitori

@odooNextev odooNextev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

L'ho usato per migrare l10n_it_vat_registries e funziona

Ti vorrei solo segnalare che nel frattempo il modulo account_tax_balance è stato pubblicato quindi puoi rimuovere i test-requirements: https://github.com/OCA/account-financial-reporting/tree/18.0/account_tax_balance

In più (anche se non ne so bene il motivo) nelle linee guida direbbero di rimuovere il file credits: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0#tasks-to-do-in-the-migration

@Borruso Borruso force-pushed the 18.0-mig-l10n_it_account branch from 35ea546 to da95108 Compare March 21, 2025 09:20
@Borruso

Borruso commented Mar 21, 2025

Copy link
Copy Markdown
Contributor Author

L'ho usato per migrare l10n_it_vat_registries e funziona

Ti vorrei solo segnalare che nel frattempo il modulo account_tax_balance è stato pubblicato quindi puoi rimuovere i test-requirements: https://github.com/OCA/account-financial-reporting/tree/18.0/account_tax_balance

In più (anche se non ne so bene il motivo) nelle linee guida direbbero di rimuovere il file credits: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0#tasks-to-do-in-the-migration

test-requirements rimossi
rimuovere il file credits che file è??

@odooNextev

Copy link
Copy Markdown
Contributor

L'ho usato per migrare l10n_it_vat_registries e funziona
Ti vorrei solo segnalare che nel frattempo il modulo account_tax_balance è stato pubblicato quindi puoi rimuovere i test-requirements: https://github.com/OCA/account-financial-reporting/tree/18.0/account_tax_balance
In più (anche se non ne so bene il motivo) nelle linee guida direbbero di rimuovere il file credits: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0#tasks-to-do-in-the-migration

test-requirements rimossi rimuovere il file credits che file è??

questo nella cartella readme: https://github.com/OCA/l10n-italy/pull/4555/files#diff-a1be204fb116f2bfd3f543c02f4fefb499858fbe9d6197644bae5a0b2395533a

@Borruso Borruso force-pushed the 18.0-mig-l10n_it_account branch from da95108 to 6c2d289 Compare March 21, 2025 09:24
tafaRU and others added 19 commits April 10, 2025 12:47
Currently translated at 100.0% (22 of 22 strings)

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

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

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

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_account/it/
We don't want XML generation to fail if a field is empty (i.e. False)
Once it's time to validate the XML, the system will produce a more
meaningful and human-readable error.
Currently translated at 100.0% (24 of 24 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_account/it/
@eLBati eLBati force-pushed the 18.0-mig-l10n_it_account branch from 7e1ab3e to dcddc19 Compare April 10, 2025 10:48
@eLBati

eLBati commented Apr 10, 2025

Copy link
Copy Markdown
Member

Aggiunto @primes2h

Quel commit non va riaggiunto, dato che esisteva già. Il problema è che il relativo codice era stato rimosso nel commit di migrazione.

Adrebbe modificato il commit di migrazione per riaggiungere il codice eliminato.

Grazie, fatto

@eLBati

eLBati commented Apr 10, 2025

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-4555-by-eLBati-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 75fc793 into OCA:18.0 Apr 10, 2025
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@primes2h primes2h left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Manco il tempo di dare l'ok... 😆

@gbtechnology

gbtechnology commented Jun 4, 2025

Copy link
Copy Markdown

da aggiungere in migrazione pec_mail -> l10n_it_pec_email

Ciao @MarcoCalcagni , ma quindi il vecchio campo pec sull'anagrafica partner dove viene gestito? Non è chiarissimo, forse mi sono perso qualche pezzo.

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.