Skip to content
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 l10n_it_riba: modificare data pagamento distinte #4297

Open
1 of 2 tasks
MaurizioPellegrinet opened this issue Jul 25, 2024 · 6 comments · Fixed by #4383
Open
1 of 2 tasks

16.0 l10n_it_riba: modificare data pagamento distinte #4297

MaurizioPellegrinet opened this issue Jul 25, 2024 · 6 comments · Fixed by #4383
Labels
14.0 enhancement needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue.

Comments

@MaurizioPellegrinet
Copy link

MaurizioPellegrinet commented Jul 25, 2024

Is your feature request related to a problem?
Problema: l'operazione di "segna come pagato" con modalità 1 o 2 genera registrazioni "settlement riba" con data attuale.
Spesso c'è l'esigenza di modificare la data.

Describe the solution you'd like
Un'idea potrebbe essere creare una registrazione "in bozza" che ci possa permettere di inserire data personalizzata prima della conferma

Additional context
Add any other context about the feature request here.
image
image

Implementazioni:

@francesco-ooops
Copy link
Contributor

@MaurizioConte che ne pensi?

@MaurizioConte
Copy link

@MaurizioConte che ne pensi?

Visti gli automatismi che ci sono dietro alle registrazioni contabili delle RIBA proporrei di richiedere al momento una data registrazione contabile impostata di default con la data del giorno.
Nota: Stessa cosa va poi fatta nella funzione ultima di indicare pagate le riba in funzione della data scadenza e li abbiamo già un wizard in cui poi aggiungere questa campo Data Registrazione Contabile da passare alla funzione che la genera.

@odooNextev
Copy link
Contributor

A dire il vero ci sarebbe già quest'azione:

image

In cui si può impostare una data:

image

@MaurizioPellegrinet
Copy link
Author

@MaurizioConte @francesco-ooops @odooNextev
Grazie Stefano, in effetti questa operazione risponde alle esigenze di @MaurizioConte (vedi PR4202 mergiata).
Tuttavia venerdì si discuteva anche con Valentina sulla possibilità di modificare proprio la data di questa registrazione (oggi non modificabile), legata al pagamento definitivo, opzione importante soprattutto per gli accrediti con MATURAZIONE DI VALUTA.

@odooNextev
Copy link
Contributor

odooNextev commented Jul 30, 2024

@MaurizioConte @francesco-ooops @odooNextev Grazie Stefano, in effetti questa operazione risponde alle esigenze di @MaurizioConte (vedi PR4202 mergiata). Tuttavia venerdì si discuteva anche con Valentina sulla possibilità di modificare proprio la data di questa registrazione (oggi non modificabile), legata al pagamento definitivo, opzione importante soprattutto per gli accrediti con MATURAZIONE DI VALUTA.

Il problema è che la registrazione del pagamento è a tutti gli effetti una registrazione contabile che ha la data contabile in sola lettura se lo stato è diverso da bozza.
Siccome mi pare sia un vincolo solo frontend, si potrebbe mettere una discriminante con un altro parametro che identifica che è una registrazione di pagamento per giudicare se impostarlo in sola lettura oltre al solo stato come è adesso.

Togliendo la sola lettura dal campo payment_ids e date si otterrebbe comunque questo errore cercando di salvare le modifiche:

image

@francesco-ooops
Copy link
Contributor

@MaurizioPellegrinet è da fare anche per la 14?

@francesco-ooops francesco-ooops added needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue. labels Nov 8, 2024
@tafaRU tafaRU added hotfix and removed hotfix labels Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
14.0 enhancement needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants