search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
142 stars 297 forks source link

[16.0] [FIX] l10n_it_fatturapa_out: attach report to a xml invoice pops an error on "report_print_menu" field #4156

Open micheledic opened 1 month ago

micheledic commented 1 month ago

Quando provi ad allegare un report ad un XML di fattura, esce un errore perchè è stata usata fields.Selection . Inoltre non funziona la domain. Con questa PR vengono fixate entrambe le cose

micheledic commented 1 month ago

andato in fail il precommit, qual è il problema?

OpenCode commented 1 month ago

andato in fail il precommit, qual è il problema?

Te lo dice la action stessa: https://github.com/OCA/l10n-italy/actions/runs/9206643473/job/25324940111#step:7:49

In generale, installa pre-commit in locale e fallo girare lì prima di pushare così vedi gli errori (molti dei quali risolve black autonomamente cambiando il codice)

francesco-ooops commented 1 month ago

@micheledic ciao, puoi aprire una issue per tracciare il problema con i passi per riprodurre?

micheledic commented 1 month ago

@micheledic ciao, puoi aprire una issue per tracciare il problema con i passi per riprodurre?

https://github.com/OCA/l10n-italy/issues/4159 ecco a te

Principalmente credo sia un problema di contesto, visto che i values dipendono dal contesto se si perde il contesto nel ping-pong di chiamate allora non riesce a validare il value