[14.0] [FIX] l10n_it_delivery_note: carrier syncing #3815
Conversation
renda-dev
commented
Jan 5, 2024
renda-dev
commented
- Set the delivery note carrier as a 'delivery carrier' instead of a partner
- Sync the carrier between the picking and the delivery note
- All the pickings carriers inside a DN must be the same
|
Hi @MarcoCalcagni, |
There was a problem hiding this comment.
grazie mille per la PR.
molto interessante come gestione, ma per essere approvata mi manca la gestione dei dati dei precendeti DdT.
dovresti fare uno script per che a esempio:
- se non c'è carrier su picking: creare un
delivery.carriere linkarlo al DdT - se c'è carrier su picking: linkare il
carrieral DdT
inoltre cancellare il flag is_carrier dal partner che a questo punto non serve più e controllare le viste tipo in questa view l10n_it_delivery_note/views/stock_delivery_note.xml dove è impostato il domain
renda-dev
force-pushed
the
14.0-fix-l10n_it_delivery_note
branch
from
73b95db
to
a20ab8d
Compare
|
Grazie per la review! @Borruso Ho aggiornato la PR in base ai tuoi consigli, fammi sapere cosa ne pensi :) |
There was a problem hiding this comment.
LGTM
ma che differenza c'è tra delivery_method_id e carrier_id?
serve dipendenza a https://github.com/OCA/delivery-carrier/tree/14.0/delivery_carrier_partner?
| } | ||
| ) | ||
|
|
||
| delivery_carrier_id = env["delivery.carrier"].create( |
There was a problem hiding this comment.
unica osservazione tu fai sempre la create del delivery.carrier in questo caso avrai vari delivery carrier con la stesso nome perché non c'è un raggruppo per partner quindi anche se il partner sul secondo DdT è lo stesso crea di nuovo il delivery.carrier
There was a problem hiding this comment.
Grazie per avermelo fatto notare, mi era totalmente sfuggito.
Ora funziona come dovrebbe, fammi sapere se l'implementazione ti piace :)
forse non è una cattiva idea |
renda-dev
force-pushed
the
14.0-fix-l10n_it_delivery_note
branch
from
a20ab8d
to
14349ed
Compare
Questa PR punta a eliminare l'utilizzo dei partner come carrier in quanto non sono tali, quindi non credo che sia molto in sintonia con le nostre intenzioni |
- Set the delivery note carrier as a 'delivery carrier' instead of a partner - Sync the carrier between the picking and the delivery note - All the pickings carriers inside a DN must be the same
renda-dev
force-pushed
the
14.0-fix-l10n_it_delivery_note
branch
from
14349ed
to
eb0c9d9
Compare
|
Moved to #3838 |
