-
-
Notifications
You must be signed in to change notification settings - Fork 303
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
[16.0] [FIX] l10n_it_vat_statement_communication: fix file doctype declaration #4357
[16.0] [FIX] l10n_it_vat_statement_communication: fix file doctype declaration #4357
Conversation
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
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
@patrickt-oforce come mai ci sono due commit? Non dovrebbe trattarsi semplicemente del FW port di #4148? |
Lo feci con due pr ecco perché |
#4317 questa è la seconda pr non era traccia nella issue |
Buondì @tafaRU se volete li schiaccio in uno solo; io ho fatto solo cherry-pick dalla #4148 e #4317 |
sì grazie, mentre lo fai ricordati di modificare il messaggio del commit rispettando le indicazioni riportate in https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message. |
091ef4f
to
ce09455
Compare
Commit uniti |
nel commit message leggo:
ma se non erro è già presente, non lo stai aggiungendo con questa PR --> puoi modificare il messaggio? Grazie. |
Avoid to output value at zero according to AdE specification In exported xml file fix the character encoding value in doctype declaration
ce09455
to
6217e56
Compare
Aggiornato |
/ocabot merge patch |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at fb77e79. Thanks a lot for contributing to OCA. ❤️ |
Forward port of #4148 and #4317