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: Approve code review
LGTM
--
https://code.launchpad.net/~camptocamp/ocb-addons/6.1-fix_1311004_account_move_line_index-afe/+merge/216820
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-addons/6.1.
--
Mailing list:
The proposal to merge lp:~camptocamp/openerp-rma/7.0-fix-1282584 into
lp:openerp-rma has been updated.
Status: Needs review = Merged
For more details, see:
https://code.launchpad.net/~camptocamp/openerp-rma/7.0-fix-1282584/+merge/207450
--
Review: Approve code review, no tests
LGTM. Thanks !
--
https://code.launchpad.net/~camptocamp/openerp-rma/7.0-crm_claim_rma-add-picking-buttons/+merge/208334
Your team OpenERP RMA is subscribed to branch lp:openerp-rma.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post
The proposal to merge
lp:~camptocamp/openerp-rma/7.0-crm_claim_rma-add-picking-buttons into
lp:openerp-rma has been updated.
Status: Needs review = Merged
For more details, see:
https://code.launchpad.net/~camptocamp/openerp-rma/7.0-crm_claim_rma-add-picking-buttons/+merge/208334
--
Just adding a small comment to say that Raphaël Valyi discovered a bug when
doing a -u crm_claim_rma with that branch (it breaks on the duplicate
constraint). We started digging to find a solution, but we didn't had time to
implement it yet.
--
Review: Needs Fixing code review, no tests
Hi,
Thanks for improving the module. I may not have the time to read all carefully,
just what I saw in a few minutes here:
* L269 : For the method _get_default_warehouse, I prefer to get the WH
through the user because take one from
Review: Approve code review
LGTM
Regards.
--
https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311509_rde/+merge/216824
Your team OpenERP RMA is subscribed to branch lp:openerp-rma.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Pedro, I proposed this to core trunk:
https://code.launchpad.net/~camptocamp/openobject-addons/trunk-refactor-po-merge-lep/+merge/216841
--
https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745
Your team Purchase Core Editors is subscribed to branch
The proposal to merge lp:~camptocamp/openerp-rma/rma-fix-lp1311520_rde into
lp:openerp-rma has been updated.
Status: Needs review = Work in progress
For more details, see:
https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311520_rde/+merge/216839
--
this depends on
https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745
--
https://code.launchpad.net/~camptocamp/sale-wkfl/7.0-dropshipping-merge-po-lep/+merge/216756
Your team Sale Core Editors is requested to review the proposed merge of
Romain Deheele - Camptocamp has proposed merging
lp:~camptocamp/openerp-rma/rma-fix-lp1311520_rde into lp:openerp-rma.
Requested reviews:
OpenERP RMA (openerprma)
Related bugs:
Bug #1311520 in OpenERP RMA: In Product Return, link Invoice with a line
without product_id raises an error
The proposal to merge lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh
into lp:account-budgeting has been updated.
Description changed to:
The object in the store condition was the wrong one. I fixed it for the correct
one + fixed the now-called function.
For more details, see:
Matthieu Dietrich @ camptocamp has proposed merging
lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into
lp:account-budgeting.
Commit message:
[FIX] fixed store condition on fields 'analytic_amount', 'analytic_real_amount'
and 'analytic_diff_amount' + fixed function called by the
Hello,
Ravi's fix was integrated into 7.0.
revno: 10007 [merge]
revision-id: m...@openerp.com-20140423100324-6stypg5uapnl1s5f
** Changed in: openobject-addons
Status: Fix Committed = Fix Released
** Changed in: openobject-addons
Importance: Undecided = Low
--
You received this bug
I also noticed that only 20-25% of process log lines have the model field
filled. Why is this?
--
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
lp:server-env-tools.
--
Mailing
Leonardo,
thanks for the review. For your questions:
1. yes, that field needs to be stored. The initial commit mentions that a sum
of the lines is used in a groupby view, which is the default view for budget
lines, coming from a project.
2. I think the optimal solution would be to retrieve
Competing fix was merged in upstream 7.0:
https://code.launchpad.net/~openerp-dev/openobject-addons/7.0-opw-596679-rgo/+merge/186294.
Setting to rejected.
--
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-lp-1210195-RDE/+merge/181227
Your team OpenERP Community Backports Team is
@Leonardo: here is a SQL version of the request.
Tell me if that's okay for you.
Regards,
Matthieu Dietrich
--
https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix-store-fields-mdh/+merge/216849
Your team Account Core Editors is requested to review the proposed merge of
** No longer affects: ocb-addons/7.0
** No longer affects: ocb-addons
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports (Addons).
https://bugs.launchpad.net/bugs/1210195
Title:
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
Stefan Rijnhart (Therp) has proposed merging
lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain
into lp:ocb-addons.
Requested reviews:
OpenERP Community Backports Team (ocb)
Related bugs:
Bug #1311628 in OpenERP Addons: Mail catchall domain parameter being
@Alexandre, Leonardo:
I went back for a classical ORM approach. I don't think a store condition's
function is called by another user than the SUPERUSER, but it's better to be
safe.
I'm still refining a YAML test as well to add to this MP.
Regards,
Matthieu
--
thanks Matthieu.
I'm also not a fan of the SQL if I don't have a gun pointed at me. Hopefully
with the test it will be easier to improve that.
--
https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix-store-fields-mdh/+merge/216849
Your team Account Core Editors is subscribed to
Hello,
Thanks for the report and the patch, it was merged in 7.0
revno: 10010 [merge]
revision-id: m...@openerp.com-20140423132552-hpnr2skutqranvsj
** Changed in: openobject-addons
Status: New = Fix Released
** Changed in: openobject-addons
Importance: Undecided = Low
--
You
The proposal to merge lp:~camptocamp/ocb-addons/7.0-fix-1309105 into
lp:ocb-addons has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1309105/+merge/216681
--
Rejecting, as merged upstream
http://bazaar.launchpad.net/~openerp/openobject-addons/7.0/revision/10010
--
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1309105/+merge/216681
Your team OpenERP Community Backports Team is requested to review the proposed
merge of
** Also affects: ocb-web
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 (Web).
https://bugs.launchpad.net/bugs/1311076
Title:
multiple popup
** No longer affects: ocb-addons/7.0
** No longer affects: ocb-addons
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports (Addons).
https://bugs.launchpad.net/bugs/1309105
Title:
Review: Approve test
--
https://code.launchpad.net/~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain/+merge/216877
Your team OpenERP Community Backports Team is requested to review the proposed
merge of
Hello,
I added the YAML test, in order to verify that the amounts are updated after a
new analytic line on the same analytic account was added.
Regards,
Matthieu
--
https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix-store-fields-mdh/+merge/216849
Your team Account Core Editors is
** Branch linked: lp:~stefan-therp/ocb-
addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain
** 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:
That was fast :)
--
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1309105/+merge/216681
Your team OpenERP Community Backports Team is requested to review the proposed
merge of lp:~camptocamp/ocb-addons/7.0-fix-1309105 into lp:ocb-addons.
--
Mailing list:
Review: Approve
--
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 :
The proposal to merge lp:~camptocamp/ocb-addons/fix-1308364-nbi into
lp:ocb-addons has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~camptocamp/ocb-addons/fix-1308364-nbi/+merge/216023
--
The proposal to merge
lp:~camptocamp/ocb-addons/ocb-7.0-fix_1311004_account_move_line_index-afe into
lp:ocb-addons has been updated.
Status: Approved = Merged
For more details, see:
Review: Approve code review, no test
Thanks Alexandre, looks good now.
--
https://code.launchpad.net/~camptocamp/ocb-addons/6.1-fix_1311004_account_move_line_index-afe/+merge/216820
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-addons/6.1.
--
Mailing list:
** Changed in: ocb-addons/7.0
Status: Fix Committed = Fix Released
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports (Addons).
https://bugs.launchpad.net/bugs/1308364
Title:
[7.0]
Stefan Rijnhart (Therp) has proposed merging
lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain
into lp:ocb-addons.
Requested reviews:
Lara (Therp) (lfreeke): test
OpenERP Community Backports Team (ocb)
Related bugs:
Bug #1311628 in OpenERP Community
** Changed in: ocb-addons/7.0
Status: Fix Committed = Fix Released
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports (Addons).
https://bugs.launchpad.net/bugs/1311004
Title:
missing
Review: Approve code review, no test
Thanks, you saved some kittens
--
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:
Ruchir Shukla(BizzAppDev) has proposed merging
lp:~therp-nl/ocb-web/7.0-form_destroy_buttons into lp:ocb-web.
Requested reviews:
OpenERP Community Backports Team (ocb)
Related bugs:
Bug #1311076 in OpenERP Community Backports (Web): multiple popup weird
behavior
Just add a test to check invoice refund action
--
https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311509_rde/+merge/216824
Your team OpenERP RMA is subscribed to branch lp:openerp-rma.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Alex Comba - Agile BG has proposed merging
lp:~agilebg/openerp-product-attributes/adding_product_supplier_info_7 into
lp:openerp-product-attributes.
Commit message:
[ADD] module product_supplier_info
Requested reviews:
Product Core Editors (product-core-editors)
For more details, see:
Review: Needs Fixing
Ruchir, thanks for taking this dangling branch of Holger and propose it. I
assume you have tested it thoroughly. To qualify for a merge in OCB, you need
to prepare a branch that proposes the same change into upstream openerp-web.
Setting to needs-fixing to indicate that
Review: Approve code review, no test
Ok LGTM
In view all commands are array of 3 elements:
http://bazaar.launchpad.net/~openerp/openerp-web/7.0/view/head:/addons/web/static/src/js/view_form.js#L3446
--
https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311509_rde/+merge/216824
Your
Review: Approve
--
https://code.launchpad.net/~camptocamp/ocb-server/ocb-7.0-fix_1204920-afe/+merge/208390
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-server.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
I see the upstream branch is green on runbot, and Holger's old comment has been
honoured.
Thanks!
--
https://code.launchpad.net/~camptocamp/ocb-server/ocb-7.0-fix_1204920-afe/+merge/208390
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-server.
--
Mailing list:
** Changed in: openobject-addons
Status: Fix Committed = Fix Released
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports (Addons).
https://bugs.launchpad.net/bugs/1180773
Title:
[7.0]
Review: Approve code review
LGTM.
Regards.
--
https://code.launchpad.net/~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain/+merge/216877
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-addons.
--
Mailing list:
The proposal to merge lp:~camptocamp/ocb-server/ocb-7.0-fix_1204920-afe into
lp:ocb-server has been updated.
Status: Needs review = Merged
For more details, see:
https://code.launchpad.net/~camptocamp/ocb-server/ocb-7.0-fix_1204920-afe/+merge/208390
--
Review: Approve
Approve as per http://en.wikipedia.org/wiki/Polish_z%C5%82oty.
--
https://code.launchpad.net/~camptocamp/ocb-server/7.0-fix_1281290-afe/+merge/215612
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-server.
--
Mailing list:
Review: Approve
--
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix_1281290-afe/+merge/215614
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-addons.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
** Changed in: ocb-server/7.0
Status: Fix Committed = Fix Released
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports (Server).
https://bugs.launchpad.net/bugs/1204920
Title:
[trunk]
The proposal to merge lp:~camptocamp/ocb-addons/7.0-fix_1281290-afe into
lp:ocb-addons has been updated.
Status: Needs review = Merged
For more details, see:
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix_1281290-afe/+merge/215614
--
The proposal to merge lp:~camptocamp/ocb-server/7.0-fix_1281290-afe into
lp:ocb-server has been updated.
Status: Needs review = Merged
For more details, see:
https://code.launchpad.net/~camptocamp/ocb-server/7.0-fix_1281290-afe/+merge/215612
--
** Changed in: ocb-server/7.0
Status: Fix Committed = Fix Released
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports (Addons).
https://bugs.launchpad.net/bugs/1281290
Title:
Wrong
** Changed in: ocb-addons/7.0
Status: Fix Committed = Fix Released
--
You received this bug notification because you are a member of OpenERP
Community Backports Team, which is subscribed to OpenERP Community
Backports (Addons).
https://bugs.launchpad.net/bugs/1281290
Title:
Wrong
Review: Approve code review, no test
Thanks a lot !
--
https://code.launchpad.net/~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain/+merge/216877
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-addons.
--
Mailing list:
61 matches
Mail list logo