[16.0][l10n_it_fatturapa_out] Define value of FatturaPA XML 2.4.2.3 field

[mmaridev]

See issue #2774

[francesco-ooops]

@mmaridev questa risolve #2774 o solo parzialmente?

[mmaridev]

@francesco-ooops risolve la issue, a quanto ho testato.
cc @SirAionTech

[francesco-ooops]

@mmaridev riesci a verificare i test?

[mmaridev]

@francesco-ooops aggiornate le reference, test passati

[francesco-ooops]

@mmaridev se fixi il precommit ci siamo :)

[mmaridev]

@francesco-ooops done

[francesco-ooops]

@mmaridev inoltre squasha i commit insieme e assegnagli un messaggio in linea con https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#L1114

[mmaridev]

@francesco-ooops done

[mmaridev]

Merge & backport?

[francesco-ooops]

@mmaridev se vuoi già aprire una PR per il backport è molto gradito :)

[mmaridev]

@francesco-ooops Speravo gestisse in automatico @OCA-git-bot :-/

[francesco-ooops]

magari 😀

dai un'occhiata a https://github.com/OCA/oca-port

[SirAionTech]

Grazie della PR!
Ho fatto solo revisione del codice.

Per il messaggio del commit, puoi seguire https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#L1114?
In particolare

please check if the commit message is cut with ellipsis

Ci sarebbe anche

Avoid commits which simultaneously impact lots of modules.

ma in questo caso non vedo come si possano separare le modifiche senza lasciare qualche commit con i test rotti.

[SirAionTech]

A livello funzionale (non il mio campo) sono sempre un po' confuso dalle date nelle fatture, in questo caso perché non record.date?

[mmaridev]

Semplicemente perché è quello utilizzato per popolare DatiGeneraliDocumento.Data

l10n-italy/l10n_it_fatturapa_out/data/invoice_it_template.xml

Line 536 in 5986b18

<Data t-out="format_date(record.invoice_date)" />

[TheMule71]

invoice_date è la data del documento
date è la data di registrazione contabile

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[mmaridev]

@SirAionTech riusciamo a sbloccare questa PR? È aperta da gennaio...

[francesco-ooops]

@mmaridev cerca di formattare il nome commit / commit message come indicato qui: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst?plain=1#L1287

[SirAionTech]

@SirAionTech riusciamo a sbloccare questa PR? È aperta da gennaio...

Sì certo: quando hai risposto alla revisione #3892 (review) chiedi di aggiornarla

(vedi https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/requesting-a-pull-request-review)
così so che ci devo guardare

[TheMule71]

Sembra che la preferenza sia di usare DataScadenzaPagamento invece di DataRiferimentoTerminiPagamento e GiorniTerminiPagamento.

Anche Odoo standard si adegua:
https://github.com/odoo/odoo/blob/6a497a26a3b1d00a08cb30e164a7d7e8499d6d61/addons/l10n_it_edi/data/invoice_it_template.xml#L198-L206

[mmaridev]

@mmaridev cerca di formattare il nome commit / commit message come indicato qui: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst?plain=1#L1287

ho riletto brevemente le guidelines e mi sembra in linea; dimmi semplicemente come lo vuoi modificato che lo faccio

[francesco-ooops]

@mmaridev cerca di formattare il nome commit / commit message come indicato qui: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst?plain=1#L1287

ho riletto brevemente le guidelines e mi sembra in linea; dimmi semplicemente come lo vuoi modificato che lo faccio

Il link mi sembra chiaro:

When you open a PR, please check if the commit message is cut with ellipsis. For example:

.. code-block::

[FIX] module_foo: and this is my very long m[...]

When this happens, it means your message is too long. You should shorten it.
Start by rephrasing and keeping the summary very synthetic.
The explanation or motivation should be kept in the description of the commit.

[francesco-ooops]

/ocabot rebase

[OCA-git-bot]

@francesco-ooops The rebase process failed, because command git rebase origin/16.0 failed with output:

Rebasing (1/1)
Auto-merging .pre-commit-config.yaml
Auto-merging l10n_it_fatturapa_out/data/invoice_it_template.xml
CONFLICT (content): Merge conflict in l10n_it_fatturapa_out/data/invoice_it_template.xml
Auto-merging l10n_it_fatturapa_out_di/tests/data/IT06363391001_00001.xml
Auto-merging l10n_it_fatturapa_out_di/tests/data/IT06363391001_00002.xml
Auto-merging l10n_it_fatturapa_out_di/tests/data/IT06363391001_00003.xml
error: could not apply c3781d3b4... [FIX][l10n_it_fatturapa_out] Define 2.4.2.3 field
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply c3781d3b4... [FIX][l10n_it_fatturapa_out] Define 2.4.2.3 field

[mmaridev]

/ocabot rebase

[OCA-git-bot]

Sorry @mmaridev you are not allowed to rebase.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

[mmaridev]

Immagino i tests siano stati aggiornati da altra PR e per questo falliscano

[SirAionTech]

Immagino i tests siano stati aggiornati da altra PR e per questo falliscano

Al momento in questa PR falliscono sia pre-commit che i test:

L'ultima esecuziona della CI su 16.0 non ha trovato problemi:

(da https://github.com/OCA/l10n-italy/actions/runs/11437860369)

Quindi questa PR è da modificare per far passare almeno pre-commit e i test.