[16.0][MIG] l10n_it_fatturapa_out_dn: Migration to 16.0#5154
Conversation
TheMule71
force-pushed
the
16.0-mig-l10n_it_fatturapa_out_dn
branch
from
82db26c to
9efa782
Compare
|
/ocabot migration l10n_it_fatturapa_out_dn |
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
Veramente le due PR sono andate avanti in parallelo, tant'è che questa è stata creata un anno prima che l'altra venisse mergiata. I commit sono stati portati a mano.
Comunque si può simulare una migrazione e riportare le modifiche:
Originally posted by @TheMule71 in #4054 (comment)
Grazie di aver convertito la PR in una migrazione!
Purtroppo quello che hai scritto può capitare quando si aggiungono le stesse modifiche a più versioni contemporaneamente.
In questa migrazione vedo che è tutto nel commit [MIG], puoi separare le modifiche fatte automaticamente da pre-commit da quelle necessarie per la migrazione? Ti consiglio di usare https://github.com/OCA/odoo-module-migrator per creare il branch della migrazione e poi sostituire l'ultimo commit con quello che hai in questa PR.
Ad esempio la conversione dei file da .rst a .md viene fatta in automatico, quindi non dovrebbe essere nel commit [MIG].
This module adds data provided by the l10n_it_delivery_note module to XML files produced by the l10n_it_fatturapa_out module.
Currently translated at 100.0% (7 of 7 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_fatturapa_out_dn Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_fatturapa_out_dn/it/
TheMule71
force-pushed
the
16.0-mig-l10n_it_fatturapa_out_dn
branch
from
9efa782 to
e3283d1
Compare
|
Vero. Curioso. Ho rilanciato il mio script da zero. E il commit c'è. E ho anche provato Il commit [MIG] dovrebbe contenere tutte le modifiche fatte in fase di migrazione (tranne pre-commit che sono in un commit a parte). In questo caso però le modifiche dipendono da un tuo commit per cui lo sposto prima. |
|
Ho indagato un po' e non l'ho squashato ma ad un certo punto era sparito comunque. Comunque adesso dovrebbe essere mergiabile. |
monen17
left a comment
monen17
left a comment
There was a problem hiding this comment.
Grazie!
Per me i commit e il codice vanno bene, lascio fare le prove a chi userà davvero il modulo
| e_invoice_delivery_note_lines = e_invoice_lines.filtered( | ||
| lambda l: l.delivery_note_id == delivery_note | ||
| ) | ||
|
|
||
| def _filterfn(li, delivery_note=delivery_note): | ||
| return li.delivery_note_id == delivery_note | ||
|
|
||
| e_invoice_delivery_note_lines = e_invoice_lines.filtered(_filterfn) |
There was a problem hiding this comment.
Si poteva anche lasciare la lambda con qualcosa tipo:
e_invoice_delivery_note_lines = e_invoice_lines.filtered(
lambda li, dn=delivery_note: li.delivery_note_id == dn
)Ma anche così va bene 👍
| self.set_sequences(10, invoice_date) | ||
| # we don't want to really depend on defaults for the name, here | ||
| invoice.name = "INV/2019/08/0001" |
monen17
changed the title
|
/ocabot merge minor |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 61b49af. Thanks a lot for contributing to OCA. ❤️ |
6 participants
vd #4054
vd #4054 (review)
Fixes: #4005