OCA / l10n-italy

Odoo Italian localization

https://www.odoo-italia.org

GNU Affero General Public License v3.0

147 stars 301 forks source link

[FIX] l10n_it_fatturapa: remove pdf conversion of XML preview #4045

Closed odooNextev closed 5 months ago

odooNextev commented 5 months ago

4039

odooNextev commented 5 months ago

@OCA/local-italy-maintainers

eLBati commented 5 months ago

/ocabot merge patch

OCA-git-bot commented 5 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-4045-by-eLBati-bump-patch, awaiting test results.

OCA-git-bot commented 5 months ago

Congratulations, your PR was merged at 27c4cd68875c313fc8b2c61ef037d0102e12ac1e. Thanks a lot for contributing to OCA. ❤️

TheMule71 commented 5 months ago

Non andrebbe allineata anche la 14? Se decidiamo che non va invocata _run_wkhtmltopdf() dico.

odooNextev commented 5 months ago

Non andrebbe allineata anche la 14? Se decidiamo che non va invocata _run_wkhtmltopdf() dico.

Nella issue avevo scritto "no" nella v14 inteso come "non si presenta il problema" però effettivamente si può allineare lo stesso.

primes2h commented 5 months ago

Non andrebbe allineata anche la 14? Se decidiamo che non va invocata _run_wkhtmltopdf() dico.

IMHO, se non strettamente necessario per uno specifico bug, eviterei di introdurre modifiche che possono impattare negativamente a livello funzionale. Gli utenti sono abituati a trovarsi il PDF, comodo per la stampa e per avere il file già pronto per una eventuale conservazione.