Simone Rubino

Results 280 comments of Simone Rubino

Ho provato in runboat e confermo che il problema si verifica in `16.0` con il file https://github.com/OCA/l10n-italy/blob/1c0e8323ec675550e84129bdf9292ccd5962fb6d/l10n_it_fatturapa_in/tests/data/IT05979361218_001.xml: ![image](https://github.com/OCA/l10n-italy/assets/143788994/dc738056-d05a-4b77-acd2-1e39fb4fbdee) (da http://oca-l10n-italy-16-0-012e8a0d9779.runboat.odoo-community.org/web#id=19&cids=1&menu_id=140&active_id=4&model=account.move&view_type=form)

> > Puoi aggiungere un test? Così si evitano regressioni. Esiste già un test che crea il file, magari puoi semplicemente controllare che abbia l'encoding come te lo aspetti. >...

> > Grazie della PR! Puoi modificare il messaggio del commit ![image](https://private-user-images.githubusercontent.com/143788994/345465972-41edacaa-d06f-4c4c-9c78-b5da57aab689.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1OTU1NjksIm5iZiI6MTcyMDU5NTI2OSwicGF0aCI6Ii8xNDM3ODg5OTQvMzQ1NDY1OTcyLTQxZWRhY2FhLWQwNmYtNGM0Yy05Yzc4LWI1ZGE1N2FhYjY4OS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzEwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcxMFQwNzA3NDlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1lNjI0ZTA1NGQ0ZGI5ZDZjYjNmMGZiYjg0Y2JmZDZiMjVmZjI2ZDY4ZWUxMDI4Y2M1M2E2MzA4YmNjZTA3YzQxJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.gWnGIVNrYKSeEwgcfRi7Gy2hMCrPyurMmyXENv1d-wM) in modo che segua le linee guida https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message? In particolare > > > > > please check if...

> > /ocabot merge patch > > perfavore si potrebbe riprovare? :+1: /ocabot merge patch

> @OCA/local-italy-maintainers c'è una procedura per anonimizzare un XML o un sito dove verificarne la correttezza che possiamo passare a @DavideSantos ? Procedure automatiche per anonimizzare non ne conosco. Per...

> allora ho provato a passare l'xml al interno della verifica ed la fa inviare tranquillamente. Bene, allora adesso ti rimane da 1. anonimizzare il file 2. verificare che passi...

Il test `test_16_xml_import` importa con dettaglio minimo: https://github.com/OCA/l10n-italy/blob/0aa4def826188d5f5ec3eab9bae0839ec8cbee75/l10n_it_fatturapa_in/tests/test_import_fatturapa_xml.py#L443-L444 lo imposta sul partner prima di importare la fattura, puoi fare allo stesso modo. Oppure, forse sarebbe più semplice > aggiungere un...

> @SirAionTech ho aggiunto il test e ci dovremmo essere ora, ho anche fatto il porting sulla pr 16.0 Grazie, potresti includere il test nello stesso commit della sua correzione?...

> it's not that simple unfortunately, because as it is, the horizontal display only works for the PDF export, but it should also be horizontal in the preview and the...

> I'll probably never understand why people consider 'it was wrong in the past' a valid reason to do it wrong now. Me neither, and I don't think that is...