-
-
Notifications
You must be signed in to change notification settings - Fork 298
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
[12.0] [FIX][l10n_it_account] Fix: better management of account_balance_sign… #1585
[12.0] [FIX][l10n_it_account] Fix: better management of account_balance_sign… #1585
Conversation
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.
Grazie!
Riesci mica ad aggiungere un test per coprire anche quando effettivamente ci sono padri e segni
https://codecov.io/gh/OCA/l10n-italy/compare/65a57c9e6335773fb1309ddbcadd683aa9719ce1...68a425d113e6b6b14a45eb6a5876ba0e63987d0f/diff
?
@eLBati non mi è ben chiaro cosa dovrei fare, relativamente ai test... |
@SilvioGregorini mancherebbe un test per |
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.
Ah però in questo momento non si può installare l10n_it_account
su un DB con i conti già creati: SilvioGregorini#1
@eLBati grazie della PR, non mi ero accorto che ci fosse il blocco. |
@SilvioGregorini potresti rimuovere il commit di merge con un rebase (non so se ocabot lo faccia) |
…s upon account types and account groups
…le installation when accounts are already created (otherwise installation is blocked)
50220c2
to
75a1cc5
Compare
@eLBati fatto 👍 |
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.
ok
This PR has the |
/ocabot merge minor |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 231fc65. Thanks a lot for contributing to OCA. ❤️ |
…s upon account types and account groups
Collegato a #1583
--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing