Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/multi-company
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on how
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/multi-company
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on how
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/product-variant
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on how
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/product-variant
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on how
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/product-variant
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on how
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/purchase-reporting
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/purchase-reporting
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/purchase-reporting
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/purchase-reporting
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/purchase-reporting
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/sale-financial
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on how
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/sale-financial
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/sale-financial
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on how
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/vertical-medical
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/vertical-medical
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/vertical-medical
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/vertical-medical
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Review: Resubmit
Hello,
The management of the project has moved to Github:
https://github.com/OCA/vertical-medical
Please migrate your merge proposal to Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
for an explanation on
Hello,
The management of the project has moved to Github:
https://github.com/OCA/vertical-medical
If you feel this change is still needed, please migrate your merge proposal to
Github. You may want to check
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
@hurrinco: this won't be merged on Launchpad, as the project is now managed on
github. Please migrate the MP to a github PR (cf. Pedro's comment above)
--
https://code.launchpad.net/~agilebg/purchase-wkfl/imp_product_by_supplier/+merge/225197
Your team Purchase Core Editors is subscribed to
The proposal to merge
lp:~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe into
lp:~openerp-community-reviewer/openerp-mrp-reports/7.0 has been updated.
Status: Needs review = Rejected
For more details, see:
submitted to github
--
https://code.launchpad.net/~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe/+merge/209044
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:~openerp-community-reviewer/openerp-mrp-reports/7.0.
--
Mailing list:
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-reporting
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~jean-lelievre/stock-logistic-report/stock_extra/+merge/182063
Your team Stock and Logistic
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-reporting
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~jean-lelievre/stock-logistic-report/stock_batch_move/+merge/182060
Your team Stock and
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-reporting
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~jean-lelievre/stock-logistic-report/logistics_list_report/+merge/182038
Your team Stock and
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-workflow
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~agilebg/stock-logistic-flows/adding_product_customer_code_picking/+merge/202472
Your team
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-workflow
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~camptocamp/stock-logistic-flows/7.0-picking_dispatch_picking_oriented_use-rde/+merge/215147
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-workflow
Could you resubmit this MP on the new site?
--
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-workflow
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~agilebg/stock-logistic-flows/adding_picking_line_description_7/+merge/219800
Your team Stock
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-workflow
Could you resubmit this MP on the new site?
--
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-warehouse
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-location/+merge/223880
Your team
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-warehouse
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-hierarchical/+merge/223882
Your
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-warehouse
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~numerigraphe-team/stock-logistic-warehouse/7.0-add-stock-available/+merge/220758
Your team
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-warehouse
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~numerigraphe-team/stock-logistic-warehouse/7.0-fix-decimal-precision/+merge/223881
Your
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-warehouse
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~camptocamp/stock-logistic-warehouse/7.0-improve_stock_location_ownership-afe/+merge/210859
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-warehouse
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~numerigraphe-team/stock-logistic-warehouse/7.0-add-stock-available-mrp/+merge/220764
Your
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-warehouse
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~numerigraphe-team/stock-logistic-warehouse/7.0-add-stock-available-sale/+merge/220761
Your
Review: Resubmit
The source code management for this project has been moved to
https://github.com/OCA/stock-logistics-warehouse
Could you resubmit this MP on the new site?
--
https://code.launchpad.net/~camptocamp/stock-logistic-warehouse/7.0-port-base_product_merge/+merge/162121
Your team
https://github.com/odoo/odoo/pull/829
--
https://code.launchpad.net/~cedric-lebrouster/ocb-server/ocb-7.0-bug-1253052-parent-order/+merge/209708
Your team OpenERP Community Backports is subscribed to branch lp:ocb-server.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post
Alexandre Fayolle - camptocamp has proposed merging
lp:~camptocamp/account-budgeting/7.0-fix_red_test-afe into lp:account-budgeting.
Requested reviews:
Account Core Editors (account-core-editors)
Related bugs:
Bug #1334605 in Account - Budgeting: budget: tests are red
https
Review: Needs Fixing code review, no tests
please add automated tests.
Additionally I'm interested in an explanation about the various comparison
operators used : sometimes you use strict inequality, sometime -or-equal
comparison (e.g. line 241 vs line 248).
--
Review: Approve code review, test
--
https://code.launchpad.net/~agilebg/account-consolidation/7.0-bug-1296740-elbati/+merge/212487
Your team Account Core Editors is subscribed to branch
lp:account-consolidation/7.0.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to
Review: Needs Fixing code review, no test
please add an automated test which exercises the new code, so that we can at
least easily check that it does not crash.
--
https://code.launchpad.net/~camptocamp/account-financial-report/7.0-add-reference-field-xls-mdh/+merge/214905
Your team Account
Review: Needs Fixing code review, no tests
1. please run flake8 on the code and fix the issues. There are lots of long
lines, missing/superfluous spaces, as well as unused imports.
2. please add an automated test which will generate a report and ensure at
least some coverage of the module.
Review: Needs Information code review, no tests
I find your naming very confusing. Could you clarify what is space and what
is namespace here?
--
https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-account_journal_report_xls-namespace-extra/+merge/220527
Your team Account
Review: Needs Fixing
resubmit there are conflict markers in the diff
--
https://code.launchpad.net/~sebastien.beau/hr-timesheet/hr-timesheet-fix-project-id-not-required/+merge/224268
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:hr-timesheet.
--
Mailing list:
Review: Needs Fixing code review, no tests
Never used it, code looks shaggy, and there are not tests.
Are there known issues?
In it's current state, this module is not on par with the standards of OCA and
it needs care and love.
If someone has use for this module and is volunteering to
setting the MP as work in progress. Please set it back to needs review when
requested changes are made.
--
https://code.launchpad.net/~alejandrosantana/account-financial-tools/7.0-account-financial_tools--add--account_account_extended_search/+merge/200092
Your team OpenERP Community
Review: Needs Fixing code review, no tests
If people are willing to maintain this, then I'm not opposed to merging in
server-env-tools.
2 points, though, to gain my approval:
* the various size constraints seem overzealous to me (esp. server URL and
domain which are obviously too short for
Review: Needs Fixing
I would really love to see this code exercised by a couple of yaml tests...
--
https://code.launchpad.net/~pedro.baeza/account-financial-report/6.1-balance-reporting/+merge/201166
Your team Account Report Core Editors is subscribed to branch
Review: Approve code review, no tests
LGTM
--
https://code.launchpad.net/~agilebg/account-invoice-report/7_fix_1221570/+merge/223781
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-invoice-report.
--
Mailing list:
Review: Needs Fixing tried to install and run the tests
This looks like a great module. However it needs some work before further
review is done:
1. this module will cause a crash at server startup if it is present in the
addons path (not necessarily installed) and the connector module is not
Review: Needs Information code review, no tests
Could you update the description of the module to make it clear that there is
no magic done in the module.
Suggestion:
'description': Defines a user to be used for automatic action
This module adds a field 'automatic_action_user_id'
on the
The proposal to merge
lp:~numerigraphe-team/ocb-addons/7.0-fix-regression-fill-inventory-OOM into
lp:ocb-addons has been updated.
Status: Needs review = Approved
For more details, see:
Review: Approve code review, sometest without being able to exercise that code
LGTM : the connection must be closed.
But
I cannot find by reading the code a case where this branch is executed (i.e. a
logical path in which we get len(self._connections) = self._maxconn so I'm
wondering if we
thanks for the precision.
--
https://code.launchpad.net/~cedric-lebrouster/ocb-server/ocb-7.0-bug-1322191-db_maxconn/+merge/220666
Your team OpenERP Community Backports is subscribed to branch lp:ocb-server.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Alexandre Fayolle - camptocamp has proposed merging
lp:~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe into
lp:openerp-product-attributes.
Requested reviews:
Product Core Editors (product-core-editors)
For more details, see:
https://code.launchpad.net/~camptocamp/openerp
github: https://github.com/odoo/odoo/issues/372
--
You received this bug notification because you are a member of Account
Core Editors, which is subscribed to a duplicate bug report (1149601).
https://bugs.launchpad.net/bugs/1070418
Title:
budget position should support view accounts
Status
https://github.com/odoo/odoo/issues/385
--
You received this bug notification because you are a member of OpenERP
Community Backports, which is a bug assignee.
https://bugs.launchpad.net/bugs/1108228
Title:
action_stock_invoice_onshipping missing on stock.picking
Status in OpenERP Community
https://github.com/odoo/odoo/issues/389
--
You received this bug notification because you are a member of Account
Core Editors, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1146923
Title:
General account should not be required on analytic entries
Status in OpenERP
https://github.com/odoo/odoo/issues/397
--
You received this bug notification because you are a member of OpenERP
Community Backports, which is a bug assignee.
https://bugs.launchpad.net/bugs/1181291
Title:
[Trunk/7.0] Refund payment: Error occurred while validating the
field(s)
I've fixed Lorenzo's remarks, and moved the migration script to an init method
as the migration script may not be executed on initial installation
--
https://code.launchpad.net/~camptocamp/sale-wkfl/product_procurement_type/+merge/222111
Your team Sale Core Editors is subscribed to branch
Alexandre Fayolle - camptocamp has proposed merging
lp:~camptocamp/sale-wkfl/product_procurement_type into lp:sale-wkfl.
Requested reviews:
Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c): code review + no
tests
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c): code review
Review: Needs Fixing
according to Alexis, this needs fixing
--
https://code.launchpad.net/~camptocamp/account-financial-tools/7.0-fix-l10n_fr_siret-fix-lp1320918-rde/+merge/220090
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-financial-tools.
--
Mailing
Review: Approve
--
https://code.launchpad.net/~camptocamp/account-financial-tools/7.0-fix-l10n_fr_siret-fix-lp1320918-rde/+merge/220090
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-financial-tools.
--
Mailing list:
Review: Approve code review, no test
LGTM thanks!
--
https://code.launchpad.net/~camptocamp/account-financial-tools/7.0-fix-l10n_fr_siret-fix-lp1320918-rde/+merge/220090
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-financial-tools.
--
Mailing list:
Review: Approve code review, no test
LGTM
thanks for fixing this
--
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1319109/+merge/219473
Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Review: Needs Fixing code review, no test
This fix will not work on Windows.
out_filename is opened on line 11 of the file, and then the pdf converter will
try to open a file with the same name while the file is still open in OpenERP
which will fail on windows
The proposal to merge lp:~camptocamp/server-env-tools/7.0-monitoring into
lp:server-env-tools has been updated.
Status: Needs review = Work in progress
For more details, see:
https://code.launchpad.net/~camptocamp/server-env-tools/7.0-monitoring/+merge/215138
--
thanks for the review, Daniel. I'm setting this back to WIP until I get the
time for take your comments into account.
--
https://code.launchpad.net/~camptocamp/server-env-tools/7.0-monitoring/+merge/215138
Your team Server Environment And Tools Core Editors is subscribed to branch
the thing is, it is different to store a birthday (which should still be
machine parseable) and a birthdate.
When the field is called birthdate, I expect a date.
--
https://code.launchpad.net/~invitu/ocb-server/7.0/+merge/169658
Your team OpenERP Community Backports is subscribed to branch
The proposal to merge lp:~camptocamp/openerp-rma/rma-fix-lp1311509_rde into
lp:openerp-rma has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311509_rde/+merge/216824
--
Review: Approve code review, no test
LGTM
--
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 :
Review: Approve code review, no test
LGTM
--
https://code.launchpad.net/~akretion-team/hr-timesheet/70-fix-timesheet-task-bug1316456/+merge/218364
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:hr-timesheet.
--
Mailing list:
Review: Approve code review, no test
LGTM
--
https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745
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, no test
LGTM
--
https://code.launchpad.net/~camptocamp/sale-wkfl/7.0-dropshipping-merge-po-lep/+merge/216756
Your team Sale Core Editors is subscribed to branch lp:sale-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Review: Approve code review, no test
LGTM
--
https://code.launchpad.net/~camptocamp/ocb-addons/ocb-7.0-fix_1302630_document_search_order_by-rde/+merge/214486
Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons.
--
Mailing list:
Review: Needs Fixing code review, test
This patch breaks the yaml_import tests.
Extract from the logs:
TEST test_ocb70 openerp.tools.yaml_import: 2. Duplicate the parent category and
verify that the children have been duplicated too and are below the new parent
ERROR test_ocb70
The proposal to merge lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into
lp:purchase-wkfl has been updated.
Status: Needs review = Work in progress
For more details, see:
https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745
--
Review: Approve code review, test
LGTM
--
https://code.launchpad.net/~camptocamp/stock-logistic-flows/7.0-picking_dispatch_wave-according-defined-number-of-sales_rde/+merge/214568
Your team Stock and Logistic Core Editors is subscribed to branch
lp:stock-logistic-flows.
--
Mailing list:
Review: Approve code review, no test
LGTM
--
https://code.launchpad.net/~camptocamp/lp-community-utils/nag-nameerror-1312682/+merge/217216
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:lp-community-utils.
--
Mailing list:
The proposal to merge
lp:~camptocamp/openerp-product-attributes/7.0-partner-attributes-lep into
lp:openerp-product-attributes has been updated.
Description changed to:
add module partner_custom_attributes
For more details, see:
Review: Approve code review, no test
--
https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix-store-fields-mdh/+merge/216849
Your team Account Core Editors is subscribed to branch lp:account-budgeting.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
following the comment by Alexis on 23/04/2014 I'm setting this MP as work in
progress
--
https://code.launchpad.net/~akretion-team/openerp-rma/rma-cleanup-cleanup-cleanup/+merge/215970
Your team OpenERP RMA is subscribed to branch lp:openerp-rma.
--
Mailing list:
Review: Approve code review, no test
LGTM, code wise.
--
https://code.launchpad.net/~akretion-team/account-financial-report/70-fix-bug-1274194-webkit-xls/+merge/217063
Your team Account Report Core Editors is subscribed to branch
lp:account-financial-report.
--
Mailing list:
The proposal to merge lp:~akretion-team/openerp-rma/rma-cleanup-cleanup-cleanup
into lp:openerp-rma has been updated.
Status: Needs review = Work in progress
For more details, see:
https://code.launchpad.net/~akretion-team/openerp-rma/rma-cleanup-cleanup-cleanup/+merge/215970
--
Actually I had completely goofed the branches (uploaded my ocb branch to
openobject-addons and proposed to merge there, hence the changes related to ocb
present in the old MP).
Should be fixed now. Thanks for the heads up, Yannick.
--
Review: Approve code review, no test
--
https://code.launchpad.net/~camptocamp/openerp-rma/7.0-fix-1282584/+merge/207450
Your team OpenERP RMA is subscribed to branch lp:openerp-rma.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Alexandre Fayolle - camptocamp has proposed merging
lp:~camptocamp/ocb-addons/6.1-fix_1311004_account_move_line_index-afe into
lp:ocb-addons/6.1.
Requested reviews:
OpenERP Core Team (openerp)
Guewen Baconnier @ Camptocamp (gbaconnier-c2c): code review
Yannick Vaucher @ Camptocamp
Review: Needs Fixing code review, no test
Your SQL request maybe works but it is not written in an efficient way (2
DISTINCT, 1 subquery). This should be written with EXISTS.
And, each time you bypass the ORM to write raw SQL, $DEITY kills a kitten.
$DEITY also maims a puppy if you don't
** Also affects: ocb-web
Importance: Undecided
Status: New
** Also affects: ocb-web/7.0
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports
Alexandre Fayolle - camptocamp has proposed merging
lp:~camptocamp/ocb-web/7.0-fix_1190236_opw-595981-msh into lp:ocb-web.
Requested reviews:
OpenERP Community Backports Team (ocb)
Related bugs:
Bug #1190236 in OpenERP Community Backports (Web): Kanban view doesn't show
constraint error
Alexandre Fayolle - camptocamp has proposed merging
lp:~camptocamp/ocb-addons/ocb-7.0-fix_1311004_account_move_line_index-afe into
lp:ocb-addons.
Requested reviews:
Guewen Baconnier @ Camptocamp (gbaconnier-c2c): code review
Related bugs:
Bug #1311004 in OpenERP Community Backports (Addons
Review: Approve code review, test
LGTM
--
https://code.launchpad.net/~camptocamp/purchase-report/purchase_print_button/+merge/216024
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Review: Approve code review, no test
--
https://code.launchpad.net/~camptocamp/hr-timesheet/7.0-port-hr_timesheet_holidays/+merge/162124
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:hr-timesheet.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Review: Approve code review, no test
LGTM
--
https://code.launchpad.net/~camptocamp/ocb-addons/fix-1308364-nbi/+merge/216023
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-addons.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
** Also affects: ocb-addons
Importance: Undecided
Status: New
** Also affects: ocb-addons/7.0
Importance: Undecided
Status: New
** Changed in: ocb-addons/7.0
Status: New = Fix Committed
** Changed in: openobject-addons
Status: New = Fix Committed
** Changed
Alexandre Fayolle - camptocamp has proposed merging
lp:~camptocamp/ocb-server/7.0-fix_1281290-afe into lp:ocb-server.
Requested reviews:
OpenERP Community Backports Team (ocb)
Related bugs:
Bug #1281290 in OpenERP Community Backports (Server): Wrong currency code -
Poland
https
** Also affects: openobject-server
Importance: Undecided
Status: New
** Also affects: ocb-server
Importance: Undecided
Status: New
** Also affects: ocb-addons
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of
** Also affects: ocb-addons/7.0
Importance: Undecided
Status: New
** Changed in: ocb-addons/7.0
Assignee: (unassigned) = Alexandre Fayolle - camptocamp
(alexandre-fayolle-c2c)
--
You received this bug notification because you are a member of OpenERP
Community Backports Team
1 - 100 of 278 matches
Mail list logo
