-
-
Notifications
You must be signed in to change notification settings - Fork 306
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
[14.0][IMP] l10n_it_ricevute_bancarie: Add exposure/risk and improve the management of unsolved riba #4457
base: 14.0
Are you sure you want to change the base?
[14.0][IMP] l10n_it_ricevute_bancarie: Add exposure/risk and improve the management of unsolved riba #4457
Conversation
@SirAionTech va bene che teniamo tutto in un solo commit o va gestito come una migrazione? |
6f99361
to
7a71ef9
Compare
7a71ef9
to
15bcfb8
Compare
15bcfb8
to
1514290
Compare
Secondo me devono essere portati i singoli commit come in una migrazione perché ognuno ha il suo scopo. @odooNextev ti chiedo di revisionare perché la PR e i commit portati sono tuoi, come il nuovo commit in questa PR. |
1514290
to
28b4471
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review, LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Funzionale ok (testato da @stefano-ooops)
@SirAionTech giusto un dubbio, la descrizione dei commit "l10n_it_riba" va modificata con il nome v14 del modulo o lasciamo così come da commit originale? |
Si va nelle aree grigie 😄 secondo me ci andrebbe il nome del modulo che si sta effettivamente modificando nel commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test Funzionale: OK
…k as past due" wizard
28b4471
to
6d9c241
Compare
Back port di tutti i cambiamenti fatti nella PR #3984. Con piccoli riadattamenti per la 14.0