Ciao banane,
you have to use
from openerp.tools.translate import _ (see for instance
https://github.com/OCA/account-invoice-reporting/blob/7.0/invoice_webkit/invoice.py#L52
)
Anyway, the project has moved to github. Please open an issue (or better a PR)
at https://github.com/OCA/account-invoice
Done at https://github.com/OCA/purchase-reporting/pull/1
--
https://code.launchpad.net/~agilebg/purchase-report/7_fix_1331394/+merge/223799
Your team Purchase Core Editors is subscribed to branch
lp:~purchase-core-editors/purchase-report/7.0.
--
Mailing list: https://launchpad.net/~openerp-comm
Review: Disapprove
moved to https://github.com/OCA/account-invoicing/pull/9
--
https://code.launchpad.net/~agilebg/account-invoicing/adding_invoice_line_no_picking_name_7/+merge/220289
Your team Account Core Editors is subscribed to branch
lp:~account-core-editors/account-invoicing/7.0.
--
Mai
Review: Disapprove
moved to https://github.com/OCA/sale-workflow/pull/9
--
https://code.launchpad.net/~agilebg/sale-wkfl/adding_sale_line_quantity_properties_based_7/+merge/223538
Your team Sale Core Editors is requested to review the proposed merge of
lp:~agilebg/sale-wkfl/adding_sale_line_quan
PR for v8 at https://github.com/OCA/partner-contact/pull/22
--
https://code.launchpad.net/~akretion-team/partner-contact-management/base-location-geonames-import/+merge/214564
Your team Partner and Contact Core Editors is subscribed to branch
lp:~partner-contact-core-editors/partner-contact-manag
Review: Abstain
Setting WIP because this will need refactoring after
https://github.com/eLBati/sale-workflow/tree/7.0_add_price_propreties_based
--
https://code.launchpad.net/~agilebg/sale-wkfl/adding_sale_line_quantity_properties_based_7/+merge/223538
Your team Sale Core Editors is requested to
The proposal to merge
lp:~agilebg/sale-wkfl/adding_sale_line_quantity_properties_based_7 into
lp:sale-wkfl has been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~agilebg/sale-wkfl/adding_sale_line_quantity_properties_based_7/+merge/2235
Moved to https://github.com/OCA/account-consolidation/pull/3
--
https://code.launchpad.net/~agilebg/account-consolidation/7.0-fix-1334645-elbati/+merge/225266
Your team Account Core Editors is subscribed to branch
lp:~account-core-editors/account-consolidation/7.0.
--
Mailing list: https://laun
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~therp-nl/partner-contact-management
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~dr.clearcorp/partner-contact-manage
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~yannick-buron/partner-contact-manag
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~akretion-team/partner-contact-manag
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~camptocamp/partner-contact-manageme
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~therp-nl/partner-contact-management
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-co
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~akretion-team/partner-contact-manag
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-co
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~camptocamp/partner-contact-manageme
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-co
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-co
Review: Resubmit
This project is now hosted on https://github.com/OCA/partner-contact. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-co
Done
https://github.com/OCA/sale-reporting/pull/13
--
https://code.launchpad.net/~agilebg/sale-reports/7_fix_1315445/+merge/223795
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:~sale-core-editors/sale-reports/7.0.
--
Mailing list: https://launchpad.net/~openerp-comm
On 07/07/2014 03:00 PM, Leonardo Pistone - camptocamp wrote:
> maybe you could refactor a bit putting
>
> parent_parallel_tax_code_id = False
>
> in an else: block a few lines down? otherwise, lgtm
Done
--
https://code.launchpad.net/~agilebg/account-consolidation/7.0-fix-1334645-elbati/+merge/22
lep, right. I looked at it better and saw that if parent_id is missing, I
should just not fill the parent_id field of the parallel account.
--
https://code.launchpad.net/~agilebg/account-consolidation/7.0-fix-1334645-elbati/+merge/225266
Your team Account Core Editors is subscribed to branch
lp:
I confirm.
Note that the bug only occurs when you create a new tax code without parent.
--
https://code.launchpad.net/~agilebg/account-consolidation/7.0-fix-1334645-elbati/+merge/225266
Your team Account Core Editors is subscribed to branch
lp:account-consolidation/7.0.
--
Mailing list: https:/
The proposal to merge
lp:~agilebg/hr-timesheet/hr_attendance_analysis-imp-time_difference-message
into lp:hr-timesheet has been updated.
Description changed to:
This allows, when time diff errors are raised, to identify the raising
attendance
For more details, see:
https://code.launchpad.net/
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/hr-timesheet/hr_attendance_analysis-imp-time_difference-message
into lp:hr-timesheet.
Requested reviews:
HR Core Editors (hr-core-editors)
For more details, see:
https://code.launchpad.net/~agilebg/hr-timesheet
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/account-consolidation/7.0-fix-1334645-elbati into
lp:account-consolidation/7.0.
Requested reviews:
Account Core Editors (account-core-editors)
Related bugs:
Bug #1334645 in Account - Consolidation: "account_parallel_cur
"Sorry Lorenzo didn't had the time to look closely at this MP again. Can we
continue this on github ?"
Gladly!
Here it is
https://github.com/OCA/account-invoice-reporting/pull/2
--
https://code.launchpad.net/~agilebg/account-invoice-report/7.0-fix_invoice_webkit_description_bugs/+merge/205204
Y
Review: Approve
--
https://code.launchpad.net/~agilebg/purchase-wkfl/imp_product_by_supplier/+merge/225197
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists
Review: Resubmit
wrong target
--
https://code.launchpad.net/~camptocamp/account-consolidation/7.0-fix_1334639-afe/+merge/224629
Your team Account Core Editors is requested to review the proposed merge of
lp:~camptocamp/account-consolidation/7.0-fix_1334639-afe into
lp:account-consolidation.
--
The proposal to merge
lp:~agilebg/sale-wkfl/adding_sale_line_quantity_properties_based_7 into
lp:sale-wkfl has been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~agilebg/sale-wkfl/adding_sale_line_quantity_properties_based_7/+merge/2235
Review: Approve
Thanks Alex, just 1 diff remark
Diff comments:
> === added directory 'sale_line_quantity_properties_based'
> === added file 'sale_line_quantity_properties_based/__init__.py'
> --- sale_line_quantity_properties_based/__init__.py 1970-01-01 00:00:00
> +
> +++ sale_line_quant
Hello Alexis, many thanks for the module.
What do you think about creating the res.country.state records if they don't
exist, before mapping them in the 'states' dictionary?
The current version is supposed to correctly work with states if you first
create states data by modules like l10n_fr_sta
2014-06-22 23:10 GMT+02:00 Alex Comba - Agile BG :
> Lorenzo,
>
> please have a look at my last commit, now it should be ok.
>
>
Thanks Alex,
maybe we could handle even more particular cases by using the address_get
method to retrieve the company?
--
https://code.launchpad.net/~agilebg/stock-lo
Review: Needs Fixing
Including 1 diff comment
About the module name: in general I agree with Pedro.
In this case, the other modules follow the same naming as this: see
http://bazaar.launchpad.net/~account-core-editors/account-invoicing/7.0/files/47/product_customer_code_invoice/
So, for now, I wo
"I am asking myself about adding an object "storage.configuration" and a field
"storing_configuration_id" on the model "ir.model.fields". The idea will to use
a default storing configuration for every field (on file sytem for exemple) but
after we will be able to customise on every field from th
Hello Sébastien,
some PEP8 improvements here
https://code.launchpad.net/~agilebg/server-env-tools/binary_field_pep8/+merge/223935
--
https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91
Your team Server Environment And Tools Core Editors is subscribed to branch
Review: Approve code review
--
https://code.launchpad.net/~agilebg/stock-logistic-report/7_fix_1331543/+merge/223878
Your team Stock and Logistic Core Editors is subscribed to branch
lp:stock-logistic-report.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : ope
Review: Needs Fixing
should be
--
https://code.launchpad.net/~agilebg/sale-reports/7_fix_1331541/+merge/223871
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:sale-reports.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-com
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/purchase-report/7_fix_1331394 into lp:purchase-report.
Requested reviews:
Purchase Core Editors (purchase-core-editors)
Related bugs:
Bug #1331394 in Purchase - Reports: "[7.0] purchase_order_webkit -
'Descript
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/sale-reports/7_fix_1315445 into lp:sale-reports.
Requested reviews:
Sale Core Editors (sale-core-editors)
Related bugs:
Bug #1315445 in Sales - Reports: "[7.0] sale_order_webkit - 'Description'
field overlaps th
The proposal to merge lp:~agilebg/sale-reports/7_fix_1315445 into
lp:sale-reports has been updated.
Description changed to:
Same as
https://code.launchpad.net/~agilebg/account-invoice-report/7.0-fix_invoice_webkit_description_bugs/+merge/205204
For more details, see:
https://code.launchpad.net
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/account-invoice-report/7_fix_1221570 into lp:account-invoice-report.
Requested reviews:
Account Core Editors (account-core-editors)
Related bugs:
Bug #1221570 in Account - Invoicing Reports: "[7.0] invoice_webkit overw
Hello Yannick and Vincent,
I renamed list_invoice_table to list_main_table.
As you saw, in order to fix bug 1277024 we had to remove the nested tables but
this reintroduced the bug 1231329.
In order to fix bug 1231329, I moved the formatted_note within the 'nobreak'
div containing line.name
Se
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/purchase-report/7_fix_cannot_import_name_purchase_order into
lp:purchase-report.
Requested reviews:
Purchase Core Editors (purchase-core-editors)
For more details, see:
https://code.launchpad.net/~agilebg/purchase-report
The proposal to merge
lp:~agilebg/purchase-report/7_fix_cannot_import_name_purchase_order into
lp:purchase-report has been updated.
Description changed to:
[FIX] purchase-report-7.0/purchase_order_webkit/__init__.py", line 31, in
from . import purchase_order
ImportError: cannot import nam
Review: Needs Fixing code review
Hello Romain and Alexandre,
diff comments + some PEP8 remarks
product_procurement_type/product.py:27:1: E302 expected 2 blank lines, found 1
product_procurement_type/product.py:34:1: E302 expected 2 blank lines, found 1
product_procurement_type/product.py:99:13:
Review: Approve code review
--
https://code.launchpad.net/~nicolariolini/account-invoicing/add_module_account_payment_term_month/+merge/212400
Your team Account Core Editors is subscribed to branch lp:account-invoicing.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to
On 06/04/2014 08:18 AM, Sergio Corato wrote:
> In everyway, I overwritten the function (instead of overriden) and cover
> the case of 'days' too (for other purposes).
>
>
> 2014-06-04 8:12 GMT+02:00 Sergio Corato :
>
>> Ciao Lorenzo,
>> they aren't mutually exclusive, in this way only one of them i
On 06/03/2014 10:35 PM, Sergio Corato wrote:
> Hi Nicola, Lorenzo, Holger,
> what about delete "commercial_month" field and put this code instead?
> "if line.months != 0:"
>
> And in the view set the field "days" required if "months" if false and
> viceversa:
>
>
>
> and
>
>
> {'required':[
Ah, sure. It is so already.
See
http://bazaar.launchpad.net/~agilebg/hr-timesheet/fix-hr_attendance_analysis-bracket/view/69/hr_attendance_analysis/hr_attendance.py#L401
By writing the 'action' field without changing it, I'm forcing the
'_get_attendance_duration' to be executed.
I'm doing this in
Pedro, right, thanks.
I added the conversion to UTC while searching for attendances within the
current_date.
This way, if selected current_date is 29/05/2014 (for a user in GMT+2), the
sign in at 28/05/2014 22:12 UTC will be included in current_date.
So, every datetime is always expressed in the
You have been requested to review the proposed merge of
lp:~agilebg/hr-timesheet/fix-hr_attendance_analysis-report-timezone into
lp:hr-timesheet.
For more details, see:
https://code.launchpad.net/~agilebg/hr-timesheet/fix-hr_attendance_analysis-report-timezone/+merge/221374
[FIX] hr_attendance_
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/hr-timesheet/fix-hr_attendance_analysis-report-timezone into
lp:hr-timesheet.
Requested reviews:
HR Core Editors (hr-core-editors)
Alex Comba - Agile BG (tafaru)
For more details, see:
https://code.launchpad.net/~agilebg/hr
The proposal to merge
lp:~agilebg/hr-timesheet/fix-hr_attendance_analysis-bracket into
lp:hr-timesheet has been updated.
Description changed to:
[FIX] wrong opening bracket due to PEP8 restyle:
http://bazaar.launchpad.net/~hr-core-editors/hr-timesheet/7.0/revision/40.2.21
[ADD] button
Hello Pedro,
what do you mean by store triggers?
I'm triggering recomputing because 'action' is in the store dictionary:
'hr.attendance': (_get_attendances, ['name', 'action', 'employee_id'], 20),
--
https://code.launchpad.net/~agilebg/hr-timesheet/fix-hr_attendance_analysis-bracket/+merge/221278
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/hr-timesheet/fix-hr_attendance_analysis-bracket into
lp:hr-timesheet.
Requested reviews:
Alex Comba - Agile BG (tafaru)
HR Core Editors (hr-core-editors)
For more details, see:
https://code.launchpad.net/~agilebg/hr-timesheet
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/hr-timesheet/fix-hr_attendance_analysis-reference-is-not-defined
into lp:hr-timesheet.
Requested reviews:
HR Core Editors (hr-core-editors)
For more details, see:
https://code.launchpad.net/~agilebg/hr-timesheet/fix
Ciao Nicola.
Just refactored the compute method and added tests with
https://code.launchpad.net/~agilebg/account-invoicing/add_module_account_payment_term_month_improvements/+merge/221124
--
https://code.launchpad.net/~nicolariolini/account-invoicing/add_module_account_payment_term_month/+merge/2
Hello Pedro and Yannick,
I made the changes.
Thanks
--
https://code.launchpad.net/~agilebg/hr-timesheet/imp_hr_attendance_analysis_roundings/+merge/206493
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:hr-timesheet.
--
Mailing list: https://launchpad.net/~openerp-com
Hello Pedro,
just modified the name_search.
Thanks
--
https://code.launchpad.net/~agilebg/purchase-wkfl/imp_product_by_supplier/+merge/219017
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Review: Approve code review
--
https://code.launchpad.net/~agilebg/account-invoicing/adding_invoice_line_no_picking_name_7/+merge/220289
Your team Account Core Editors is requested to review the proposed merge of
lp:~agilebg/account-invoicing/adding_invoice_line_no_picking_name_7 into
lp:accou
Review: Approve code review
--
https://code.launchpad.net/~agilebg/stock-logistic-flows/adding_picking_line_description_7/+merge/219800
Your team Stock and Logistic Core Editors is requested to review the proposed
merge of lp:~agilebg/stock-logistic-flows/adding_picking_line_description_7
into
Review: Approve code review
--
https://code.launchpad.net/~agilebg/openerp-manufacturing/adding_mrp_production_properties_7/+merge/219559
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:openerp-manufacturing.
--
Mailing list: https://launchpad.net/~openerp-community
Review: Approve code review
--
https://code.launchpad.net/~pedro.baeza/purchase-wkfl/7.0-purchase_discount/+merge/217030
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-
Ok, thanks.
What about the
amount_untaxed, amount_tax, amount_total
fields repeated 2 times?
--
https://code.launchpad.net/~pedro.baeza/purchase-wkfl/7.0-purchase_discount/+merge/217030
Your team Purchase Core Editors is requested to review the proposed merge of
lp:~pedro.baeza/purchase-wkfl/7.0
Review: Needs Fixing
Thanks Pedro,
line 349: I think discount should not be < -100 too
line 399: amount_untaxed and following fields are present 2 times
some PEP8:
purchase_discount/__openerp__.py:21:1: O603 Manifest "license" key is missing
purchase_discount/__openerp__.py:29:5: O600 Warning u
Dear Yannick,
I don't think this MP would break compatibility with
account_financial_report_webkit.
I installed account_financial_report_webkit and account_parralel_currency and I
can use them without problems.
It just doesn't copy the centralized field to the parallel accounts. For that,
a li
Review: Needs Fixing
line 35: missing description
--
https://code.launchpad.net/~savoirfairelinux-openerp/purchase-wkfl/pallet-delivery-missing-security-access/+merge/218508
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
--
Mailing list: https://launchpad.net/~openerp
Review: Needs Fixing
Thanks Vincent,
some style fixes:
account_invoice_analytic_required/__init__.py:3:2: W291 trailing whitespace
account_invoice_analytic_required/__init__.py:19:75: W291 trailing whitespace
account_invoice_analytic_required/__init__.py:23:14: E271 multiple spaces after
keywor
Review: Approve code review
--
https://code.launchpad.net/~numerigraphe-team/purchase-wkfl/7.0-add-purchase_delivery_split_date/+merge/211374
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/purchase-wkfl/imp_product_by_supplier into lp:purchase-wkfl.
Requested reviews:
Purchase Core Editors (purchase-core-editors)
For more details, see:
https://code.launchpad.net/~agilebg/purchase-wkfl/imp_product_by_supplier/+merge
Review: Approve code review
--
https://code.launchpad.net/~agilebg/purchase-wkfl/7.0-bug-1312024-tafaru/+merge/217061
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-rev
Review: Approve code review
Thanks
--
https://code.launchpad.net/~akretion-team/sale-wkfl/70-add-sale_order_revision/+merge/202735
Your team Sale Core Editors is requested to review the proposed merge of
lp:~akretion-team/sale-wkfl/70-add-sale_order_revision into lp:sale-wkfl.
--
Mailing list:
Review: Approve code review
Ok :-)
--
https://code.launchpad.net/~acsone-openerp/account-financial-tools/account_partner_required-sbi/+merge/216442
Your team Account Core Editors is requested to review the proposed merge of
lp:~acsone-openerp/account-financial-tools/account_partner_required-sbi
Review: Needs Information
Thanks Alex,
why did you remove the 'images' and 'sequence' items?
--
https://code.launchpad.net/~agilebg/purchase-wkfl/7.0-bug-1312024-tafaru/+merge/217061
Your team Purchase Core Editors is requested to review the proposed merge of
lp:~agilebg/purchase-wkfl/7.0-bug-13
Review: Needs Information
Hello Stéphane,
why not using '_constraints' member instead of overriding create and write
methods?
Is there any technical reason to avoid YAML tests?
Thanks!
--
https://code.launchpad.net/~acsone-openerp/account-financial-tools/account_partner_required-sbi/+merge/21
The proposal to merge
lp:~openerp-community/hotel-management-system/porting-to-7-branch into
lp:hotel-management-system/7.0 has been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~openerp-community/hotel-management-system/porting-to-7-br
Review: Abstain
I suspect the modules contained in lp:hotel-management-system/7.0 are not ready
for v7
I would test the modules contained in
lp:~serpentcs/hotel-management-system/hotel-7.0 and possibly merge them into
lp:hotel-management-system/7.0
--
https://code.launchpad.net/~agilebg/hotel
Review: Approve code review
just remove "name = ''" at line 110
Thanks!
--
https://code.launchpad.net/~agilebg/account-invoice-report/7.0-bug-1284049-hurrinico/+merge/208753
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-invoice-report.
--
Mailing list: htt
Review: Needs Fixing
Nico,
line 74,76: I think you should use 'lot_icecream_0' and 'lot_icecream_1'
line 91: you are writing 'prodlot_id' field with another lot, but I think it's
unnecessary, as lots are already assigned by 'stock.move.split' wizard.
line 115: the test is checking if the invoi
Review: Approve code review
--
https://code.launchpad.net/~onnis-roberto/web-addons/web-addons/+merge/214906
Your team Web-Addons Core Editors is subscribed to branch lp:web-addons.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@list
Review: Approve code review
--
https://code.launchpad.net/~agilebg/account-invoicing/adding_stock_invoice_picking_incoterm_7/+merge/213987
Your team Account Core Editors is subscribed to branch lp:account-invoicing.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to
Review: Approve
Thanks to openerp-conventions
https://github.com/nbessi/openerp-conventions
I see
delivery_optional_invoice_line/__openerp__.py:43:5: O600 Warning unknown
Manifest key ('active')
Apart from this, LGTM
--
https://code.launchpad.net/~agilebg/carriers-deliveries/adding_delivery_opt
On 04/01/2014 03:28 PM, Guewen Baconnier @ Camptocamp wrote:
>> Actually I prefer not to delete the MP in order not to lose the discussion. I
>> prefer to set it 'rejected', but I dont' have the rights, so I wait for a
>> community reviewer.
> Done.
Thanks :-)
--
https://code.launchpad.net/~agi
Review: Approve code review
--
https://code.launchpad.net/~acsone-openerp/account-financial-tools/allow_date_fy-sbi/+merge/213644
Your team Account Core Editors is requested to review the proposed merge of
lp:~acsone-openerp/account-financial-tools/allow_date_fy-sbi into
lp:account-financial-t
Actually I prefer not to delete the MP in order not to lose the discussion. I
prefer to set it 'rejected', but I dont' have the rights, so I wait for a
community reviewer.
--
https://code.launchpad.net/~agilebg/server-env-tools/add_textual_database_name/+merge/209184
Your team Server Environmen
Review: Approve code review
--
https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-base_delivery_carrier_label-tracking_ids/+merge/207199
Your team Stock and Logistic Core Editors is subscribed to branch
lp:carriers-deliveries.
--
Mailing list: https://launchpad.net/~openerp-commun
Review: Approve code review
--
https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-delivery_carrier_label_postlogistics-tracking_ids/+merge/207202
Your team Stock and Logistic Core Editors is subscribed to branch
lp:carriers-deliveries.
--
Mailing list: https://launchpad.net/~opene
Review: Needs Fixing
Hello Guewen,
just a little fix
line 39: trackings is actually a list of browse records
Thanks
--
https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-delivery_carrier_label_postlogistics-tracking_ids/+merge/207202
Your team Stock and Logistic Core Editors is subsc
Review: Disapprove
Hello Thomas,
what is the purpose of this merge proposal?
Please take a look at https://doc.openerp.com/contribute/ about how to
contribute.
Thanks
--
https://code.launchpad.net/~hr-core-editors/hr-timesheet/6.1/+merge/213140
Your team HR Core Editors is subscribed to branch l
"I don't know why I thought it didn't work"
I suspect it was due to bug 1268510
--
https://code.launchpad.net/~agilebg/server-env-tools/add_textual_database_name/+merge/209184
Your team Server Environment And Tools Core Editors is requested to review the
proposed merge of lp:~agilebg/server-env-
Review: Disapprove
Actually, this module does just the same as setting
list_db = False
I don't know why I thought it didn't work...
This module is useless
--
https://code.launchpad.net/~agilebg/server-env-tools/add_textual_database_name/+merge/209184
Your team Server Environment And Tools Core Ed
See discussion at
https://code.launchpad.net/~icsergio/account-payment/commercial-year-payment/+merge/210700
--
https://code.launchpad.net/~nicolariolini/account-invoicing/add_module_account_payment_term_month/+merge/212400
Your team Account Core Editors is requested to review the proposed merge o
Lorenzo Battistini - Agile BG has proposed merging
lp:~agilebg/account-consolidation/7.0-bug-1296740-elbati into
lp:account-consolidation/7.0.
Requested reviews:
Account Core Editors (account-core-editors)
Related bugs:
Bug #1296740 in Account - Consolidation: "7.0 account_parallel_cur
** Also affects: openobject-addons
Importance: Undecided
Status: New
** Changed in: openobject-addons
Status: New => Fix Committed
** Also affects: ocb-addons
Importance: Undecided
Status: New
** Changed in: ocb-addons
Status: New => Fix Released
** Changed in:
On 03/21/2014 05:51 PM, Yannick Vaucher @ Camptocamp wrote:
> mrp_repair_webkit/__init__.py:5:80: E501 line too long (95 > 79 characters)
> mrp_repair_webkit/__openerp__.py:5:80: E501 line too long (95 > 79 characters)
> mrp_repair_webkit/report/__init__.py:5:80: E501 line too long (95 > 79
> char
Review: Needs Fixing
Hello Alexandre,
just checked with
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/checkout-pep8
$ ./checkout-flake8.sh
lp:~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe
mrp_repair_webkit
Making a temp directory
branching
Review: Approve test
--
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/checkout-pep8/+merge/211098
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:lp-community-utils.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to
1 - 100 of 179 matches
Mail list logo