l10n-italy icon indicating copy to clipboard operation
l10n-italy copied to clipboard

[ADD] l10n_it_vat_settlement_date

Open SirAionTech opened this issue 11 months ago • 6 comments

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

SirAionTech avatar Mar 20 '24 08:03 SirAionTech

@TheMule71 questa è la migrazione della tua PR #2208, interessa?

SirAionTech avatar Mar 20 '24 10:03 SirAionTech

@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 avatar Mar 20 '24 13:03 TheMule71

@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.

SirAionTech avatar Mar 20 '24 14:03 SirAionTech

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 avatar Mar 22 '24 09:03 SirAionTech

@SirAionTech puoi rimuovere il file per i test, visto che la #1137 è stata mergiata? Grazie.

sergiocorato avatar Mar 22 '24 15:03 sergiocorato

@SirAionTech puoi rimuovere il file per i test, visto che la #1137 è stata mergiata? Grazie.

Fatto

SirAionTech avatar Mar 22 '24 15:03 SirAionTech

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). 🤖

OCA-git-bot avatar May 20 '24 08:05 OCA-git-bot

merge?

andreampiovesana avatar May 20 '24 12:05 andreampiovesana

/ocabot merge nobump

eLBati avatar May 20 '24 14:05 eLBati

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.

OCA-git-bot avatar May 20 '24 14:05 OCA-git-bot

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

OCA-git-bot avatar May 20 '24 14:05 OCA-git-bot