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

14.0 mig l10n it delivery note base #1962

Closed

Conversation

fredzamoabg
Copy link
Contributor

@fredzamoabg fredzamoabg commented Nov 27, 2020

Descrizione del problema o della funzionalità:
migrazioone l10n_it_delivery_note_base e l10n_it_delivery_note

Comportamento attuale prima di questa PR:

Comportamento desiderato dopo questa PR:

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

@OCA-git-bot
Copy link
Contributor

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

@fredzamoabg fredzamoabg mentioned this pull request Nov 27, 2020
76 tasks
@TheMule71
Copy link
Contributor

Ciao, in previsione della migrazione alla 14, ho fatto un POC su come aggiungere i dati delle delivery note all'XML (simulando la funzionalità del modulo attuale l10n_it_fatturaout_ddt per l10n_it_ddt).

Il risultato è qui:
https://github.com/TheMule71/l10n-italy/tree/12.0-efattura-out-qweb-dn

In sostanza, si crearà un modulo ponte nel quale mettere i template e test. io qui mi sono limitato a fare una prova e ho integrato l'xml direttamente nel modulo.

Ci sono cose ancora da valutare (as es. il riferimento al numero di riga non va messo se il DDT si riferisce all'intera fattura).

Ho modificato un test e test passa (vanno aggiunte tutte le info obbligatorie per la generazione dell'XML).

@MarcoCalcagni
Copy link
Contributor

alcune correzioni da fare :

  1. nella tree delle delivery note stock.delivery.note.tree la variabile state è tradotta come provincia dovrebbe essere stato
  2. se faccio un trasferimento tra 2 magazzini interni non mi permette di creare il ddt.

per il resto va bene ho fatto vari test

@MarcoCalcagni
Copy link
Contributor

/ocabot merge

@OCA-git-bot
Copy link
Contributor

Hi @As400it. Your command failed:

Required option bumpversion_mode for command merge. Possible values : major, minor, patch, nobump.

Ocabot commands

  • ocabot merge major|minor|patch|nobump

More information

@fredzamoabg
Copy link
Contributor Author

create 2 PR per splittare i 2 moduli:

#2140
#2141

questa PR può essere chiusa

@primes2h primes2h closed this Feb 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants