[14.0][FIX] l10n_it_delivery_note shipping address #3989
Conversation
|
Hi @MarcoCalcagni, @renda-dev, @aleuffre, |
There was a problem hiding this comment.
La modifica è ok a livello tecnico.
il titolo è fuorviante: la modifica fa in modo che, come contatto ricevente su un DdT creato da un SO, viene inserito l'indrizzo di fatturazione, e non il contatto principale. (Questo è comunque diverso dall'indirizzo di spedizione, che è un campo a parte sul DdT)
Non so dire a livello funzionale quale sia l'indirizzo corretto da inserire.
sergiocorato
force-pushed
the
14.0-fix-l10n_it_delivery_note-shipping_address
branch
from
e151ae2 to
ad73eaf
Compare
|
Questa PR va a toccare lo stesso argomento di #3975 |
sergiocorato
force-pushed
the
14.0-fix-l10n_it_delivery_note-shipping_address
branch
from
ad73eaf to
b61fccd
Compare
Grazie delle note, ho corretto il titolo della issue e la descrizione. Questa PR è il port di un'altra PR già aperta da qualche anno sulla 12.0, la #2773 . Ha alcuni punti di contatto con la #3975 , ma questa fixa un bug, mentre quella aggiunge delle informazioni. Il fatto che un DDT debba essere intestato alla stessa ditta che poi riceverà la fattura è la norma da quanto ne so, non mi immagino che una ditta x riceva una fattura da y e che alleghi i DDT intestati a z (con una destinazione qualsiasi). |
sergiocorato
force-pushed
the
14.0-fix-l10n_it_delivery_note-shipping_address
branch
from
b61fccd to
9ca3f81
Compare
sergiocorato
force-pushed
the
14.0-fix-l10n_it_delivery_note-shipping_address
branch
from
9ca3f81 to
414f5ab
Compare
|
/ocabot merge minor |
|
This PR looks fantastic, let's merge it! |
|
It looks like something changed on |
|
Congratulations, your PR was merged at 8c7c115. Thanks a lot for contributing to OCA. ❤️ |
No description provided.