-
-
Notifications
You must be signed in to change notification settings - Fork 382
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
[17.0][MIG] account_statement_base: Migration to 17.0 #614
[17.0][MIG] account_statement_base: Migration to 17.0 #614
Conversation
This module contains the views of account.bank.statement.line, which have been removed from the account module in Odoo v16
The module account_statement_base will be used by the modules from OCA/bank-statement-import which are LGPL (because they initially came from Odoo official addons), so it cannot be AGPL.
* refactoring JS in order to add logic * Improve views * Minor fixes in Odoo code in order to fix all possible options
Currently translated at 100.0% (12 of 12 strings) Translation: account-reconcile-16.0/account-reconcile-16.0-account_statement_base Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_statement_base/pt_BR/
Currently translated at 100.0% (12 of 12 strings) Translation: account-reconcile-16.0/account-reconcile-16.0-account_statement_base Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_statement_base/es/
Currently translated at 100.0% (12 of 12 strings) Translation: account-reconcile-16.0/account-reconcile-16.0-account_statement_base Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_statement_base/fr/
Thanks for the PR title renaming. You just have missed the underscores on the module name. I have just put them. /ocabot migration account_statement_base |
@pedrobaeza Hi Pedro! I would like to ask you if there is something missing in this PR in order to be merge it. Thanks in advance! Best regards! |
You can review other PRs, and ask in exchange they review yours. |
Can you check the warning that Odoo emits on installation of the module?
|
Oh, fixed here I see. OCA/maintainer-tools#595 |
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.
LGTM (I installed it and opened a statement and one of its lines in form view.)
This PR has the |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at c24a979. Thanks a lot for contributing to OCA. ❤️ |
No description provided.