Simone Rubino

> @SirAionTech fatto Avevo appena fatto rebase e c'era solo da rimuovere un file, però il diff delle ultime modifiche https://github.com/OCA/l10n-italy/compare/8c584d3901419f43f383d3c64e2d43bc2c9cdc43..c4199ef1b9dd5295208976a5804c489bf5d15ac5 riporta più di 40 file modificati. Mi sembra sia...

> Hi @SirAionTech. Your command failed: > > `Invalid options for command rebase: per cercare di isolare le ultime modifiche`. > > **Ocabot commands** > > * `ocabot merge major|minor|patch|nobump`...

Ok https://github.com/OCA/l10n-italy/compare/8c584d3901419f43f383d3c64e2d43bc2c9cdc43..e4565fed3a160572f336f1048929387e3dbf408b ha più senso: include le modifiche di https://github.com/OCA/l10n-italy/pull/4434 che è stata mergiata nel frattempo, ma per il resto c'è solo il file rimosso :+1: /ocabot merge nobump

/ocabot migration l10n_it_delivery_note

Risolve #3952 per `16.0`.

> @MaurizioConte da quanto ho capito è possibile indicarne uno predefinito nel fornitore e questo viene riportato nella fattura fornitore, ma è comunque possibile selezionarne un altro nella fattura fornitore...

> > Le righe che hai modificato in `l10n_it_fatturapa_in` e `l10n_it_fatturapa_out` non sono coperte dai test, hai modo di aggiungere un semplice test per verificare che funzionino come ti aspetti?...

@francescapenso potresti mettere un titolo più specifico?

> @SirAionTech buona per te? A me già non convinceva questa parte di codice che prende tutto quello che c'è nella cache e lo scrive. Questa PR non mi sembra...

> @SirAionTech riusciamo a risalire a chi/quando era stata aggiunta questa funzione? Fu aggiunta in https://github.com/OCA/l10n-italy/commit/773e1c3f61bf8f32197010a99f8094c97b1d9c07 (il commit originale è https://github.com/OCA/l10n-italy/commit/cb063e9795bf3557a56ef58772bd0e54ce13dd90) da @eLBati