[FIX][l10n_it_vat_statement_communication] fix encoding file and remo…

[matteoopenf]

…ve zero value from xml in according the specification of ADE

[matteoopenf]

sistemo il diff e ci siamo

[matteoopenf]

LGTM, grazie Matteo!

Grazie a te

[francesco-ooops]

@SirAionTech riesci a dare un'occhiata? questa è approvata ma è rimasta arenata

[matteoopenf]

Grazie della PR! Puoi modificare il messaggio del commit 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 the commit message is cut with ellipsis

e forse la descrizione del commit potrebbe essere meno ripetitiva 😉

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.

Ho cercato in https://www.agenziaentrate.gov.it/portale/web/guest/iva-2024 e non ho trovato parti che obbligano ad includere l'encoding, ma suppongo non faccia male aggiungerlo. Ho provato a usare il software di controllo su una dichiarazione generata in runboat ma ci sta mettendo un po' tanto ad aprirsi, secondo me non gli piace Ubuntu.

si i test li ho dovuti fare poi su windows, per il test non sono praticissimo, vedo intanto per sistemare il messaggio di commit che facendo rebase si sono sommati

[matteoopenf]

commit sistemato

[matteoopenf]

Grazie della PR! Puoi modificare il messaggio del commit 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 the commit message is cut with ellipsis

e forse la descrizione del commit potrebbe essere meno ripetitiva 😉

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.

Ho cercato in https://www.agenziaentrate.gov.it/portale/web/guest/iva-2024 e non ho trovato parti che obbligano ad includere l'encoding, ma suppongo non faccia male aggiungerlo. Ho provato a usare il software di controllo su una dichiarazione generata in runboat ma ci sta mettendo un po' tanto ad aprirsi, secondo me non gli piace Ubuntu.

non trovo il test che crea il file per anche solo ipotizzare la modifica del test, per caso me lo puoi indicare?

[SirAionTech]

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.

non trovo il test che crea il file per anche solo ipotizzare la modifica del test, per caso me lo puoi indicare?

Ah hai ragione, stavo guardando in 16.0 dove c'è https://github.com/OCA/l10n-italy/blob/9f773c4fd8a757bf82d93c9a5047e476e005cdde/l10n_it_vat_statement_communication/tests/test_vat_statement_communication.py#L380. In 14.0 non c'è ancora nulla :scream:

[SirAionTech]

Grazie della PR! Puoi modificare il messaggio del commit 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 the commit message is cut with ellipsis

e forse la descrizione del commit potrebbe essere meno ripetitiva 😉

commit sistemato

Grazie, ora è meno ripetitivo :smile: però ancora non segue le linee guida, per lo stesso motivo che ho scritto sopra

[matteoopenf]

Grazie della PR! Puoi modificare il messaggio del commit 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 the commit message is cut with ellipsis

e forse la descrizione del commit potrebbe essere meno ripetitiva 😉

commit sistemato

Grazie, ora è meno ripetitivo 😄 però ancora non segue le linee guida, per lo stesso motivo che ho scritto sopra

ora?

[OCA-git-bot]

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-4148-by-SirAionTech-bump-patch, awaiting test results.

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[matteoopenf]

Visto che non esiste già nulla di simile nei test e la modifica è minima, per stavolta mergiamo senza test :smile:

/ocabot merge patch

Grazie mille

[OCA-git-bot]

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