Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0] [FIX] l10n_it_delivery_note_base: add migration for model data #3859

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

odooNextev
Copy link
Contributor

Nella versione 14.0 del modulo l10n_it_delivery_note_base sono stati rimossi degli ir.model.data per avere una creazione dinamica delle sequenze ed i tipi di DN per ogni azienda.
#3257

Questo crea un problema durante la migrazione da 12.0 a 14.0 perchè, post aggiornamento del modulo, Odoo cerca di cancellare i riferimenti a questi record perchè non esistono più nei data, ma non deve farlo perchè esistono ugualmente nel DB.

Per sistemare la questione propongo di cancellare in anticipo nel pre-migrate i riferimenti di modo che post migrazione non cerchi di rimuovere anche i record automaticamente.

@OCA-git-bot
Copy link
Contributor

Hi @MarcoCalcagni,
some modules you are maintaining are being modified, check this out!

@francesco-ooops
Copy link
Contributor

@odooNextev confermi che è da fare solo per la 14?

@odooNextev
Copy link
Contributor Author

@odooNextev confermi che è da fare solo per la 14?

Sì, perché rimuovere i data è un'operazione da fare solo una volta.

@SirAionTech

This comment was marked as resolved.

@odooNextev
Copy link
Contributor Author

Il problema non c'è anche quando si aggiorna da prima di 4299c11 a dopo quelle modifiche? Se è così la versione della migrazione dovrebbe essere 14.0.2.3.1 (da 4299c11#diff-1bb0b5543f8a8a7714cc0a80c9b03ece858fb1948f67b93a7a83cfe10f816d06R16)

è vero, ma la versione allora è 14.0.1.0.1 perché la migrazione è di l10n_it_delivery_note_base non l10n_it_delivery_note

@odooNextev odooNextev force-pushed the 14.0-fix-l10n_it_delivery_note-mig branch from da26ecb to ebd8355 Compare January 17, 2024 11:31
@SirAionTech

This comment was marked as resolved.

@stenext stenext force-pushed the 14.0-fix-l10n_it_delivery_note-mig branch from ebd8355 to e59ead6 Compare January 29, 2024 07:53
@sergiocorato
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 14.0.

@OCA-git-bot OCA-git-bot force-pushed the 14.0-fix-l10n_it_delivery_note-mig branch from e59ead6 to 84d56a5 Compare February 27, 2024 10:19
@sergiocorato
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-3859-by-sergiocorato-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2beaefd into OCA:14.0 Feb 27, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 2beaefd. 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.

5 participants