l10n-italy
l10n-italy copied to clipboard
[ADD] l10n_it_vat_settlement_date
Implementa https://github.com/OCA/l10n-italy/issues/4034 per 16.0
.
In pratica è la migrazione del modulo in https://github.com/OCA/l10n-italy/pull/2208 (che non è mai stata mergiata), ho aggiunto i test e il refactoring di qualche altro modulo (compreso account_tax_balance) per semplificare i test e poter fare degli override dove viene chiamato super
.
Dipende da:
- [x] https://github.com/OCA/account-financial-reporting/pull/1137
@TheMule71 questa è la migrazione della tua PR #2208, interessa?
@TheMule71 questa è la migrazione della tua PR #2208, interessa?
Ottimo ma ricordo che la cosa era stata cassata a livello di idea (nonostante salti fuori ciclicamente).
Io posso solo ribadire la mia posizione, già espressa in passato: ci sono N date identificabili,
- data documento (impostato da chi lo crea)
- data ricezione (quando viene consegnata nel tuo cassetto fiscale / quando odoo la scarica - se tutto va bene: potenziamente data consegna e data import potrebbe anche essere diverse se si blocca il server PEC per es.)
- data registrazione (quando viene confermata la fattura in bozza derivante dall'import di fatturapa_in)
- periodo della liquidazione IVA nel quale considerare la fattura
Per me, le prime tre (o quattro contando la data di import) sono oggettive, e andrebbero salvate separatamente nel DB con il dato reale. La quarta è una semplice selezione, il campo DB può essere una data (la famigerata "data di competenza iva") o anche una selection (periodo precedente / periodo attuale) presente se è ancora fattibile come opzione.
@TheMule71 questa è la migrazione della tua PR #2208, interessa?
Ottimo ma ricordo che la cosa era stata cassata a livello di idea (nonostante salti fuori ciclicamente).
Io posso solo ribadire la mia posizione, già espressa in passato: ci sono N date identificabili,
- data documento (impostato da chi lo crea)
- data ricezione (quando viene consegnata nel tuo cassetto fiscale / quando odoo la scarica - se tutto va bene: potenziamente data consegna e data import potrebbe anche essere diverse se si blocca il server PEC per es.)
- data registrazione (quando viene confermata la fattura in bozza derivante dall'import di fatturapa_in)
- periodo della liquidazione IVA nel quale considerare la fattura
Per me, le prime tre (o quattro contando la data di import) sono oggettive, e andrebbero salvate separatamente nel DB con il dato reale. La quarta è una semplice selezione, il campo DB può essere una data (la famigerata "data di competenza iva") o anche una selection (periodo precedente / periodo attuale) presente se è ancora fattibile come opzione.
Premetto che io dell'idea ho capito fino a un certo punto, per questo con @Marianna-Marasco abbiamo aperto preventivamente la issue https://github.com/OCA/l10n-italy/issues/4034: per valutare lì pro e contro prima di procedere all'implementazione.
Visto che dopo due settimane c'era solo la tua risposta https://github.com/OCA/l10n-italy/issues/4034#issuecomment-1985907321 (con un'implementazione per 12.0
), abbiamo pensato non ci fosse problema a procedere con l'implementazione.
Ma forse non ho capito: stai dicendo che secondo te l'idea non va bene o che c'è qualche problema con questa implementazione? Se è l'idea a non andare bene, forse sarebbe meglio parlarne nella issue; altrimenti non ho capito qual'è il problema di questa implementazione.
Ne abbiamo parlato un pochino stamattina in chiamata: aggiungo che siccome in questa implementazione viene fatto tutto in un modulo a parte, chi non vuole usare la data di competenza IVA può semplicemente non installare il nuovo modulo.
@SirAionTech puoi rimuovere il file per i test, visto che la #1137 è stata mergiata? Grazie.
@SirAionTech puoi rimuovere il file per i test, visto che la #1137 è stata mergiata? Grazie.
Fatto
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). 🤖
merge?
/ocabot merge nobump
What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-4062-by-eLBati-bump-nobump, awaiting test results.
Congratulations, your PR was merged at bd75734182eb171ff4487e8600ad0615278991a3. Thanks a lot for contributing to OCA. ❤️