-
-
Notifications
You must be signed in to change notification settings - Fork 307
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
[IMP][14.0] l10n_it_intrastat show weight total #4440
[IMP][14.0] l10n_it_intrastat show weight total #4440
Conversation
@sergiocorato se tu non fossi in @OCA/local-italy-maintainers, io metterei missing issue e goodnight; però visto che sei un maintainer secondo me dovresti fare uno di:
Oppure non ci interessa più tenere allineate le versioni? Poi se vuoi puoi provare l'ebbrezza di aggiornare la issue di sincronizzazione #4391. |
Il tempo è tiranno 😅, ho caricato la PR e sono uscito, domani faccio il resto. |
Grazie, allora finché non è pronta la metto in bozza |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ho provato allora la logica della proposta su https://github.com/OCA/l10n-italy/wiki/Team-di-Sviluppo-(proposta) (inserito qui la descrizione della modifica/correzione, aggiunto sulla issue riepilogativa la PR). |
Ok proviamo 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grazie della PR!
"Total" è più usato di "Tot", ma sopravviveremo.
/ocabot merge minor
On my way to merge this fine PR! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test funzionale: OK
This PR has the |
Congratulations, your PR was merged at c8ebcb6. Thanks a lot for contributing to OCA. ❤️ |
Semplice PR per far vedere il totale dei kg nelle liste Intrastat, come si vede il totale dell'importo.