[18.0][IMP] l10n_it_accompanying_invoice: handler for adding transport data in XML

[Borruso]

Issue: odoo-italia/task991

[eLBati]

Grazie.
Un testino? 🙏

[eLBati]

Abbiamo detto che , per i campi non obbligatori, bisogna controllare che siano valorizzati

[francesco-ooops]

@Borruso @eLBati questa non serve per 16/14 giusto?

[eLBati]

@francesco-ooops giusto

[monen17]

/ocabot rebase

[OCA-git-bot]

Congratulations, PR rebased to 18.0.

[monen17]

Grazie della PR!
In buona parte va bene, non mi torna molto però che i nuovi nodi non vengano valorizzati nel XML dei test, puoi verificare?

Puoi anche aggiornare la documentazione?

[monen17]

Perché non c'è anche MezzoTrasporto? Penso sia il campo delivery_transport_method_id

[monen17]

Qui manca

Suggested change

	<Descrizione
	<!-- 2.1.9.5 -->
	<Descrizione

[monen17]

Oltra ai nodi che ho segnalato, mancano anche:

• DataOraRitiro
• DataOraConsegna

Non so però se abbiamo il dato da poterci mettere o se lo vogliamo mettere in generale, mi rimetto al parere dei funzionali 🙏

[monen17]

La maggior parte dei nuovi nodi aggiunti non vengono inseriti nel XML anche se nel test vengono assegnati:

        italian_partner.default_transport_condition_id = self.env.ref(
            "l10n_it_delivery_note.transport_condition_PF"
        )
        italian_partner.default_goods_appearance_id = self.env.ref(
            "l10n_it_delivery_note.goods_appearance_CAR"
        )
        italian_partner.default_transport_reason_id = self.env.ref(
            "l10n_it_delivery_note.transport_reason_VEN"
        )
        italian_partner.default_transport_method_id = self.env.ref(
            "l10n_it_delivery_note.transport_method_MIT"
        )

Manca anche tutto DatiAnagraficiVettore, li puoi aggiungere?

[monen17]

Come mai questi in un nuovo file? Sei partito da qualcosa di esistente?

[tafaRU]

@monen17 @eLBati potreste aggiornare la vostra review?