-
-
Notifications
You must be signed in to change notification settings - Fork 866
[16.0] [MIG] base_comment_template #664
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] [MIG] base_comment_template #664
Conversation
Port modules base_comment_template and invoice_comment_template to V10
OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex
Currently translated at 100.0% (15 of 15 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/gl/ [ADD] icon.png
Currently translated at 100.0% (15 of 15 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/ja/
Currently translated at 100.0% (15 of 15 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/pt/
Currently translated at 100.0% (15 of 15 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/pt_BR/
[IMP] Move comment_template_id from account_invoice_comment_template and make it company_dependant [IMP] base_comment_template: Add test
…d to the Invoicing tab [IMP] account_invoice_comment_template: rename partner field name from comment_template_id to invoice_comment_template_id [IMP] account_invoice_comment_template: Make partner field company_dependant and move domain definition of invoice fields from the view to the model [MOV] account_invoice_comment_template: comment_template_id to base_comment_template [IMP] account_invoice_comment_template: Translate templates when partner changes
[UPD] README.rst Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/
Currently translated at 100.0% (21 of 21 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/pt_BR/
Currently translated at 100.0% (21 of 21 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/pt/
Currently translated at 95.2% (20 of 21 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/hr/
Previous field declaration was incorrect due to a typo, leading to non applying the company dependent features. Now everything is correct and the migration scripts care of moving data properly.
Previous field declaration was incorrect due to a typo, leading to non applying the company dependent features. Now everything is correct and the migration scripts care of moving data properly.
Currently translated at 100.0% (21 of 21 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/de/
Currently translated at 100.0% (21 of 21 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/pt_PT/
Currently translated at 100.0% (21 of 21 strings) Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/sk/ Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-invoice-reporting-12.0/account-invoice-reporting-12.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/account-invoice-reporting-12-0/account-invoice-reporting-12-0-base_comment_template/
As the field was not properly converted to property at 2.0.0, the script fails to do its job when migrating from v11, so we need to adapt the scripts to this situation.
[UPD] Update base_comment_template.pot [UPD] README.rst
base_comment_template 14.0.2.0.1
Currently translated at 100.0% (36 of 36 strings) Translation: reporting-engine-14.0/reporting-engine-14.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-14-0/reporting-engine-14-0-base_comment_template/pt/
Currently translated at 100.0% (36 of 36 strings) Translation: reporting-engine-14.0/reporting-engine-14.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-14-0/reporting-engine-14-0-base_comment_template/it/
[UPD] Update base_comment_template.pot [UPD] README.rst base_comment_template 14.0.3.0.0 Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: reporting-engine-14.0/reporting-engine-14.0-base_comment_template Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-14-0/reporting-engine-14-0-base_comment_template/
base_comment_template 14.0.3.0.1
|
/ocabot migration base_comment_template |
|
Sorry @AMhadhbi you are not allowed to mark the addon tobe migrated. To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons. If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the |
|
/ocabot migration base_comment_template |
marylla
left a comment
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.
I actually only do functional testing and this module is just the base module, but I checked in runboat that I can see the views:
- tree view of comments in partner form view
- menu for all comments here: settings > technical > reporting > comment templates
Of course, I could not create a comment template because we need account_comment_template (OCA/account-invoice-reporting#242) or sale_comment_template or a similiar extension.
I will test those after this module is merged. :)
Can someone do a code review too?
|
This PR has the |
|
@pedrobaeza Hi Pedro, can you merge this PR please? |
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 92f5706. Thanks a lot for contributing to OCA. ❤️ |
No description provided.