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

[16.0] [FIX] l10n_it_delivery_note: recipient/shipping address fixes #3609

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

odooNextev
Copy link
Contributor

Come segnalato nella issue, il campo contact dei picking viene nascosto se il trasferimento è di tipo interno o consegna, ma viene richiesto quando se ne valida uno creato manualmente (inventory > transfers > create).
#3597

Per cominciare propongo di lasciarlo sempre visibile e di impostare il partner della DN con il campo "contatto" del picking se non esiste un ordine di vendita collegato, ma si può discutere anche di soluzioni alternative.

Abbiamo anche pensato di aggiungere un campo nel picking correlato al partner della DN per mostrarlo anche nella vista embed della DN nella scheda del picking.

Abbiamo anche voluto correggere un altro comportamento indesiderato nei trasferimenti interni: una volta che si inserisce "contact" e si valida, nel report della DN viene stampato "shipping address" = indirizzo di WH2, ignorando completamente il campo "contact".
Con la modifica invece nel picking viene impostato automaticamente il campo "contact" con l'indirizzo preso dal magazzino della "Destination Location".

@OCA-git-bot
Copy link
Contributor

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

@odooNextev odooNextev changed the title [FIX] Contact field on picking always visible [16.0] [FIX] l10n_it_delivery_note: recipient/shipping address fixes Sep 25, 2023
Copy link

github-actions bot commented Jun 9, 2024

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 9, 2024
@odooNextev
Copy link
Contributor Author

odooNextev commented Aug 1, 2024

@francesco-ooops pensi non interessi più a nessuno questa PR?

@francesco-ooops
Copy link
Contributor

@odooNextev ciao, mi piacerebbe riprenderla un giorno perchè la struttura del DN è molto convoluta, ma non credo avverrà a breve

@odooNextev
Copy link
Contributor Author

@odooNextev ciao, mi piacerebbe riprenderla un giorno perchè la struttura del DN è molto convoluta, ma non credo avverrà a breve

Io dovrei fare solo un rebase, ma poi per essere approvata necessita di review.
Lo farei solo quando c'è qualcuno pronto a fare test ed approvarla nel caso.

@MaurizioPellegrinet
Copy link

@odooNextev @francesco-ooops per il test funzionale, se serve, io sono disponibile

@stenext stenext force-pushed the 16.0-fix-l10n_it_delivery_note-contact branch from efce259 to 0b7341e Compare August 1, 2024 10:17
@odooNextev
Copy link
Contributor Author

Grazie per la disponibilità @MaurizioPellegrinet
Ho fatto il rebase ed ora puoi testare la funzionalità nel runboat
Se non è esaustiva la descrizione della PR in cima e la issue #3597, chiedi pure

Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Funzionale OK

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Aug 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

l10n_it_delivery_note - “recipient” cannot be set on manual pickings
4 participants