-
-
Notifications
You must be signed in to change notification settings - Fork 665
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
[MIG] account_payment_term_extension: Migration to 11.0 #375
[MIG] account_payment_term_extension: Migration to 11.0 #375
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.
Please squash commits by logical set/author
@@ -58,30 +55,30 @@ msgstr "Nombre mostrado" | |||
#. module: account_payment_term_extension | |||
#: model:ir.ui.view,arch_db:account_payment_term_extension.view_payment_term_line_form | |||
msgid "Due Date Computation" | |||
msgstr "" | |||
msgstr "álculo de fecha de vencimiento" |
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.
Missing initial C
@@ -1,29 +1,25 @@ | |||
# Translation of Odoo Server. |
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.
Remove this file
# Copyright 2013-2016 Camptocamp SA (Yannick Vaucher) | ||
# Copyright 2004-2016 Odoo S.A. (www.odoo.com) | ||
# Copyright 2015-2016 Akretion | ||
# (Alexis de Lattre <alexis.delattre@akretion.com>) | ||
# Copyright 2018 Simone Rubino - Agile Business Group | ||
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). | ||
|
||
|
||
from functools import reduce |
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.
Why this?
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.
reduce function is used here:
https://github.com/OCA/account-invoicing/pull/375/files#diff-d0188f250f37daab85f1680d28b455ccR198
and in python3 does not exist. We have two solutions, import from functools or replace reduce function by a for loop.
I choose import option.
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, thanks for the explanations
3f06ab4
to
749b770
Compare
…_term_extension Add support for months and weeks in account.payment.term.line Add unittest PEP8 validation
Fix and add a feature for apply a chronological order on lines Merge of the 'account_payment_term_multi_day' module Add icon to module
749b770
to
96e8f18
Compare
cc @Tecnativa