[Openerp-community] [ANN] OCA code sprint in Louvain la Neuve on June 1-2, 2015

2015-04-15 Thread Alexandre Fayolle
Hello everyone,


The OCA will be organizing a code sprint / meetup in Louvain la Neuve on
the 2 days preceding Odoo Experience 2015.

You will find details on this blog post:
https://odoo-community.org/blog/news-1/post/2-day-sprint-before-odoo-experience-2015-15

including a survey that you should fill in if you plan to attend so that
we have an idea of which
tracks are drawing interest and how many people we should expect.

Thanks for your attention, and I hope to see you at the sprint, or
during Odoo Experience 2015.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Could oca adopt report_webkit before v8

2015-03-30 Thread Alexandre Fayolle
 investment take over the open source
 investment.
 Of course monetizing a public community asset (AGPL ecosystem)
 into a private differentiated asset
 (say non AGPL SaaS) is always a way to generate a some cash, just
 like when a public company is
 privatized.

 All this is investment, whether it's money or whether it is code
 open source code. And the community
 certainly deserves the same commitment with its contracts (the
 AGPL license) than your financial
 investors get with their investment contracts. After all,
 investment of OpenERP/Odoo community
 weights quite a bit more than 6 millions euros...

 So it sounds fair to ask for commitment to AGPL before community
 contribute you more free code to your ecoystem, no?

 So am just asking about that commitment and I think this is
 legitimate from somebody or community who
 participated heavily on the ecosystem investment of OpenERP/Odoo.
 Sorry if you found it offensive, but my question about commitment
 to AGPL was serious in fact.


 Regards.

 -- 
 Raphaël Valyi
 Founder and consultant
 http://twitter.com/rvalyi http://twitter.com/#%21/rvalyi
 +55 21 3942-2434 tel:%2B55%2021%203942-2434
 www.akretion.com http://www.akretion.com/



 ___
 Mailing list: https://launchpad.net/~openerp-community
 https://launchpad.net/%7Eopenerp-community
 Post to : openerp-community@lists.launchpad.net
 mailto:openerp-community@lists.launchpad.net
 Unsubscribe : https://launchpad.net/~openerp-community
 https://launchpad.net/%7Eopenerp-community
 More help   : https://help.launchpad.net/ListHelp




 ___
 Mailing list: https://launchpad.net/~openerp-community
 Post to : openerp-community@lists.launchpad.net
 Unsubscribe : https://launchpad.net/~openerp-community
 More help   : https://help.launchpad.net/ListHelp


-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Maintainers needed for aeroo reports in OCA

2015-01-15 Thread Alexandre Fayolle
On 5/01/2015 9:36 PM, Alexandre Fayolle
alexandre.fayo...@camptocamp.com
mailto:alexandre.fayo...@camptocamp.com wrote:

 Hello,

 I remember this being discussed, but I cannot find the discussion
 in my
 mail archive...

 Is someone planning on proposing aeroo reports to
 OCA/reporting-engine?
 If not, I will make PRs to remove all addons depending on aeroo in the
 OCA code base.


Hi everyone,

I wanted to raise notice here:

https://github.com/OCA/reporting-engine/issues/1 was not well received
by sraps / Alistek who is not a member of the OCA.

I don't feel like pushing things at this point and add a new module for
our code base for which we would not have a CLA.
I've therefore submitted a PR on hr to remove the modules depending on
aeroo: https://github.com/OCA/hr/pull/63

If people care about these reports, now is the time to fork the OCA/hr
repository to extract these. We can certainly add a pointer to that new
repository so that users will find the new home more easily.


-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Maintainers needed for aeroo reports in OCA

2015-01-15 Thread Alexandre Fayolle

On 15/01/2015 09:30, Pedro Manuel Baeza Romero wrote:
 Well, it's a pity to hear that. I want to push to have Aeroo OCA
 version forked from sraps, but as CLA is considered important enough,
 I'm not going to push again.


If you push hard enough, you may get through. It would not be the first
module on which the OCA does not have a full CLA, and I won't block a PR
on this ground (but I won't approve it either). Please propose for
merging and let the debate happen.

I personnally don't use aeroo reports, which means I have no incitement
to push forward on this, and I propose a simple resolution. As I feel
I'm maybe not the best placed person to handle this from end to end, I
bring the issue to the mailing list, so that people not closely
following Github are aware of it.



-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Licenses guidelines for OCA modules

2015-01-12 Thread Alexandre Fayolle

 Here we can do it better, there are some guides about this:
 # -*- coding: utf-8 -*-


 __author__ = Full Name(email)
 __copyright__ = Copyright (C) YEAR FULL NAME
 __license__ = AGPL 3.0
 __version__ = 1.0

 So in pythonic way we must add LICENSE file in each module, 
 and as best practice AUTHORS.txt and add Full Name (email) as list as make 
 contributions.

 All of module templates can implement it and help us.
  


This is something I like a lot.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


[Openerp-community] Checking the CLA on OCA repositories

2015-01-06 Thread Alexandre Fayolle
Dear community and OCA Contributors,

I have configured a web hook on all OCA project github repositories for
CLA checking. The goal is to check that contributors have signed the OCA
Contributor Licence Agreement (http://odoo-community.org/page/cla)
http://odoo-community.org/page/cla%29. 

What to do if your Pull Request is flagged by the web hook?

* if you did not send your CLA yet, it is time to do so now. Be sure to
include your scanned CLA and your github username in the mail
 (c...@odoo-community.org)

* if you did send your CLA, maybe we did not have time to process it, or
maybe you github login is not known from us. Discuss the issue with the
project leader who will come back to us and we will do the necessary
paperwork. 

We thank you for your contribution to the OCA.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


[Openerp-community] Maintainers needed for aeroo reports in OCA

2015-01-05 Thread Alexandre Fayolle
Hello,

I remember this being discussed, but I cannot find the discussion in my
mail archive...

Is someone planning on proposing aeroo reports to OCA/reporting-engine?
If not, I will make PRs to remove all addons depending on aeroo in the
OCA code base.

Best wishes for 2015 to everyone.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Webkit Reports future...

2014-09-22 Thread Alexandre Fayolle
On 21/09/2014 17:02, Lionel Sausin wrote:
 Hi,
 The situation here is :
 - merge proposal for report_webkit have been ignored by the RD team
 until after 7.0
 - RD now accepts pull requests more willingly, but 8.0 is frozen so
 no feature will go in
 - pull requests for master will also be refused because report_wekbit
 is removed
 - OCA's backport branch OCB won't help us either because, since the
 features are refused in master, they would not be backports but new
 features.

 So, it's a bit absurd but I'm afraid we're deadlocked until at least
 v9 is released.
 Unless maybe OCA can make an exception by accepting new features for
 deprecated modules?
 Lionel

I'm not willing to do that, because I'm concerned we will get issues
with some addons breaking with OCB / Official and not with the other one.

The proper way of handling this would be to propose an *addon* to OCA
adding the papersize feature / report with no margin.

Then you can install that addon or not, depending on your needs, and not
break anything.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Webkit Reports future...

2014-09-22 Thread Alexandre Fayolle
On 21/09/2014 17:02, Lionel Sausin wrote:
 Hi,
 The situation here is :
 - merge proposal for report_webkit have been ignored by the RD team
 until after 7.0
 - RD now accepts pull requests more willingly, but 8.0 is frozen so
 no feature will go in
 - pull requests for master will also be refused because report_wekbit
 is removed
 - OCA's backport branch OCB won't help us either because, since the
 features are refused in master, they would not be backports but new
 features.


I've thought about this a bit more, and I think, given the nature of the
changes which are mentioned in the 2 original PR, this could make it to
OCB. The rules are a bit hazy here, and certainly less strict than the
ones in official odoo. We do have some changes in there which were later
rejected in official...

I suggest you push a PR on the OCB branches, where they will be examined
and discussed (and eventually accepted or rejected). It will certainly
cost less than maintaining your own fork.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Reclaiming context

2014-08-29 Thread Alexandre Fayolle
On 28/08/2014 18:04, Sandy Carter wrote:
 Hello community,


 While debugging yet another translation error in Odoo 7.0, I came up
 with a simple way to reclaim lost contexts in functions.

 The bug I was fighting is one where workflow actions do not pass any
 contexts. This bug manifests as chattr messages in english when using
 the workflows.

 As I don't have the time to fix it in the server, nor am I ready to jump
 through the hoops to get that fix integreated in the Odoo, I found a way
 to fix this bug in my module.

 The way I countered the bug in my module is in the write function where
 I would usually start with

 if context is None:
 context = {}

 or

 context = context or {}

 In this case it was None since the workflow mechanism would not have a
 context. I saw that since I have the user id, I can still get the
 context for this user.

 I replaced the familiar context check with:

 context = context or self.pool['res.users'].context_get(cr, uid)


Ideally this would be done by the workflow engine, but I'm pretty sure
this won't make it to official 7.0 and not sure for OCB. But I do like
the version you propose *a lot*. Let's use it more!

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com



___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~ajite/multi-company/multi-company-add-0001 into lp:multi-company

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~ajite/multi-company/multi-company-add-0001/+merge/203454
Your team Multi Company Core Editors is subscribed to branch lp:multi-company.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/multi-company/multi-company-action-user into lp:multi-company

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~akretion-team/multi-company/multi-company-action-user/+merge/179179
Your team Multi Company Core Editors is subscribed to branch lp:multi-company.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~0s/openerp-product-variant/openerp-product-variant into lp:openerp-product-variant

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~0s/openerp-product-variant/openerp-product-variant/+merge/213423
Your team Product Core Editors is subscribed to branch 
lp:openerp-product-variant.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/openerp-product-variant/product-variant-multi-attribute into lp:openerp-product-variant

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~akretion-team/openerp-product-variant/product-variant-multi-attribute/+merge/223611
Your team Product Core Editors is subscribed to branch 
lp:openerp-product-variant.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~credativ/openerp-product-variant/product-variant-7.0-view-priority into lp:openerp-product-variant

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~credativ/openerp-product-variant/product-variant-7.0-view-priority/+merge/228704
Your team Product Core Editors is subscribed to branch 
lp:openerp-product-variant.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/purchase-report/7_fix_1331394 into lp:purchase-report

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 how to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~agilebg/purchase-report/7_fix_1331394/+merge/223799
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~jean-lelievre/purchase-report/move_reports into lp:purchase-report

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 how to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~jean-lelievre/purchase-report/move_reports/+merge/182044
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~jean-lelievre/purchase-report/product_stock_type into lp:purchase-report

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 how to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~jean-lelievre/purchase-report/product_stock_type/+merge/182050
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~jean-lelievre/purchase-report/product_inventory_warning into lp:purchase-report

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 how to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~jean-lelievre/purchase-report/product_inventory_warning/+merge/182057
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~serpentcs/purchase-report/purchase-serpentcs into lp:purchase-report

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 how to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~serpentcs/purchase-report/purchase-serpentcs/+merge/190323
Your team Purchase Core Editors is subscribed to branch lp:purchase-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/sale-financial/7.0-port-sale_line_watcher into lp:sale-financial

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~camptocamp/sale-financial/7.0-port-sale_line_watcher/+merge/214058
Your team Sale Core Editors is subscribed to branch lp:sale-financial.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/sale-financial/7.0-port-sale_markup into lp:sale-financial

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~camptocamp/sale-financial/7.0-port-sale_markup/+merge/214061
Your team Sale Core Editors is subscribed to branch lp:sale-financial.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/sale-financial/7.0-port-sale_floor_price into lp:sale-financial

2014-08-22 Thread Alexandre Fayolle - camptocamp
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 to proceed.

Thanks for contributing to the project

-- 
https://code.launchpad.net/~camptocamp/sale-financial/7.0-port-sale_floor_price/+merge/214059
Your team Sale Core Editors is subscribed to branch lp:sale-financial.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~echeverrifm/oemedical/oemedical_lifestyle into lp:oemedical

2014-08-21 Thread Alexandre Fayolle - camptocamp
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 how to proceed. 

Thanks for contributing to the project


-- 
https://code.launchpad.net/~echeverrifm/oemedical/oemedical_lifestyle/+merge/205518
Your team OEmedical Commiter is subscribed to branch lp:oemedical.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~echeverrifm/oemedical/oemedical_lab into lp:oemedical

2014-08-21 Thread Alexandre Fayolle - camptocamp
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 how to proceed. 

Thanks for contributing to the project


-- 
https://code.launchpad.net/~echeverrifm/oemedical/oemedical_lab/+merge/205519
Your team OEmedical Commiter is subscribed to branch lp:oemedical.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~echeverrifm/oemedical/oemedical_socioeconomics into lp:oemedical

2014-08-21 Thread Alexandre Fayolle - camptocamp
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 how to proceed. 

Thanks for contributing to the project


-- 
https://code.launchpad.net/~echeverrifm/oemedical/oemedical_socioeconomics/+merge/205523
Your team OEmedical Commiter is subscribed to branch lp:oemedical.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~the-clone-master/oemedical/oemedical_refactoring_step_I into lp:oemedical

2014-08-21 Thread Alexandre Fayolle - camptocamp
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 how to proceed. 

Thanks for contributing to the project


-- 
https://code.launchpad.net/~the-clone-master/oemedical/oemedical_refactoring_step_I/+merge/202240
Your team OEmedical Commiter is subscribed to branch lp:oemedical.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~jeffery9/oemedical/oemedical into lp:oemedical

2014-08-21 Thread Alexandre Fayolle - camptocamp
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 how to proceed. 

Thanks for contributing to the project


-- 
https://code.launchpad.net/~jeffery9/oemedical/oemedical/+merge/194051
Your team OEmedical Commiter is subscribed to branch lp:oemedical.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~svass/oemedical/oemedical into lp:oemedical

2014-08-21 Thread Alexandre Fayolle - camptocamp
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
 for an explanation on how to proceed. 

Thanks for contributing to the project


-- 
https://code.launchpad.net/~svass/oemedical/oemedical/+merge/141364
Your team OEmedical Commiter is subscribed to branch lp:oemedical.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/purchase-wkfl/imp_product_by_supplier into lp:~purchase-core-editors/purchase-wkfl/7.0

2014-08-20 Thread Alexandre Fayolle - camptocamp
@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 branch 
lp:~purchase-core-editors/purchase-wkfl/7.0.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe into lp:~openerp-community-reviewer/openerp-mrp-reports/7.0

2014-08-19 Thread Alexandre Fayolle - camptocamp
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:
https://code.launchpad.net/~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe/+merge/209044
-- 
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: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe into lp:~openerp-community-reviewer/openerp-mrp-reports/7.0

2014-08-19 Thread Alexandre Fayolle - camptocamp
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: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Use of new odoo API when porting OCA addons

2014-08-12 Thread Alexandre Fayolle
On 12/08/2014 09:57, Holger Brunn wrote:
  Should it be a prerequisite when proposing addons port or new
  addons on OCA project for v8 and above?

 New addons for v8: Yes, but I think that's a non-issue because people
 will be eager to use it anyways
 Ported addons: No, as I'd expect this to slow down porting efforts.
 And it's a non-trivial change, so I'd prefer to have a running version
 in the first place and migrating the API afterward.



I tend to agree with Holger.

IMO it is more important to improve test coverage than to port to the
new API.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] Track datamodel changes

2014-08-07 Thread Alexandre Fayolle
On 06/08/2014 21:16, David Arnold - El Alemán wrote:
 *Hi People*
 *
 *
 I want to keep track on any db-changes realized on the database
 coloums during this last development phase until V8 stable is released.

 I thought of something with grep and some magical statements (I surely
 would not quite understand). Does anyone have an idea how to
 accomplish this? Or is there any best practice arround?

Hi David,

There are tools for this in the OpenUpgrade project.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] odoo vertical-medical

2014-08-05 Thread Alexandre Fayolle

On 05/08/2014 09:52, Mario Arias wrote:
 Serpent (Jay ?)...  it could be interesting if you / others propose
 some more samples...  The idea of the post was to create interest from
 the group and get help on the project...   ;-)
 The original logo is attached, as well as one for odoo medical...

 Pedro, we also like OdooMedical more.  We will do the renaming, but as
 that is a second step, we have time for the community to provide more
 feedback...

 In the mean time, this is the updated logo...


 ​

 Regards,
 -Mario


If I may verge on the pedantic side... this is not the correct symbol:
you are using Hermes' caduceus (linked to commerce) and not the rod of
Asclepius' rod (linked to medecine). The rod of Asclepius does not bear
wings, and has a single snake.

See http://en.wikipedia.org/wiki/Rod_of_Asclepius for more information,
and http://en.wikipedia.org/wiki/Caduceus_as_a_symbol_of_medicine and
then choose your side :-)

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] odoo vertical-medical

2014-08-05 Thread Alexandre Fayolle
You may want to use this font for the logo:
http://www.fungamesedu.com/wp-content/uploads/2013/05/Doctors-alphabet.jpg

(sorry, could not resist)

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] github migration: proposale for helper for travis configuration

2014-07-03 Thread Alexandre Fayolle
Hello,

I did the tranfer of ownership. The correct repository is now
https://github.com/OCA/maintainer-quality-tools

Please update your remotes and configuration files :)

Alexandre


On 02/07/2014 16:24, Joël Grand-Guillaume wrote:
 +1 for including this under the OCA umbrelly in the community
 Maintainers team.


 On Wed, Jul 2, 2014 at 1:48 PM, Alexandre Fayolle
 alexandre.fayo...@camptocamp.com
 mailto:alexandre.fayo...@camptocamp.com wrote:

 Hello,

 I've spent some time setting up
 https://github.com/gurneyalex/maintainer-quality-tools

 The goal is to provide a setup for running OCA Addons tests with
 travis
 when using OpenERP 6.1 or 7.0 which don't have the odoo.py command. It
 should also be possible in the near future to add support for 8.0, and
 so as to keep a very similar .travis.yml file in the different
 branches.
 Later this project could serve as host for more tools if needed.

 An example addon using this is stock-logistic-warehouse (7.0 branch)
 (which has a failed build, but also has no tests and lots of non pep8
 compatible code... we will need to fix this)

 If people agree, I would like to put this under the OCA umbrella, and
 encourage its use for all addons projects. Maybe the porting
 documentation could point to it. PR for improvements are welcome. :-)

 Thanks for your attention.

 --
 Alexandre Fayolle
 Chef de Projet
 Tel : + 33 (0)4 79 26 57 94

 Camptocamp France SAS
 Savoie Technolac, BP 352
 73377 Le Bourget du Lac Cedex
 http://www.camptocamp.com


 ___
 Mailing list: https://launchpad.net/~openerp-community
 https://launchpad.net/%7Eopenerp-community
 Post to : openerp-community@lists.launchpad.net
 mailto:openerp-community@lists.launchpad.net
 Unsubscribe : https://launchpad.net/~openerp-community
 https://launchpad.net/%7Eopenerp-community
 More help   : https://help.launchpad.net/ListHelp




 -- 


 *camptocamp*
 INNOVATIVE SOLUTIONS
 BY OPEN SOURCE EXPERTS

 *Joël Grand-Guillaume*
 Division Manager
 Business Solutions

 +41 21 619 10 28
 www.camptocamp.com http://www.camptocamp.com/



-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~jean-lelievre/stock-logistic-report/stock_extra into lp:stock-logistic-report

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 Core Editors is subscribed to branch 
lp:stock-logistic-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~jean-lelievre/stock-logistic-report/stock_batch_move into lp:stock-logistic-report

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 Logistic Core Editors is subscribed to branch 
lp:stock-logistic-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~jean-lelievre/stock-logistic-report/logistics_list_report into lp:stock-logistic-report

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 Logistic Core Editors is subscribed to branch 
lp:stock-logistic-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/stock-logistic-flows/adding_product_customer_code_picking into lp:stock-logistic-flows

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/stock-logistic-flows/7.0-picking_dispatch_picking_oriented_use-rde into lp:stock-logistic-flows

2014-07-03 Thread Alexandre Fayolle - camptocamp
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
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/stock-logistic-flows/7.0-stock_cancel_restrict_reopen_button_to_wh_managers into lp:stock-logistic-flows

2014-07-03 Thread Alexandre Fayolle - camptocamp
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-stock_cancel_restrict_reopen_button_to_wh_managers/+merge/223529
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/stock-logistic-flows/adding_picking_line_description_7 into lp:stock-logistic-flows

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing into lp:stock-logistic-flows/6.1

2014-07-03 Thread Alexandre Fayolle - camptocamp
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/~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing/+merge/189182
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-flows/6.1.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-location into lp:stock-logistic-warehouse

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 Stock and Logistic Core Editors is requested to review the proposed 
merge of lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-location 
into lp:stock-logistic-warehouse.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-hierarchical into lp:stock-logistic-warehouse

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-warehouse.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/stock-logistic-warehouse/7.0-add-stock-available into lp:stock-logistic-warehouse

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-warehouse.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/stock-logistic-warehouse/7.0-fix-decimal-precision into lp:stock-logistic-warehouse

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-warehouse.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/stock-logistic-warehouse/7.0-improve_stock_location_ownership-afe into lp:stock-logistic-warehouse

2014-07-03 Thread Alexandre Fayolle - camptocamp
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
Your team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-warehouse.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/stock-logistic-warehouse/7.0-add-stock-available-mrp into lp:stock-logistic-warehouse

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-warehouse.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/stock-logistic-warehouse/7.0-add-stock-available-sale into lp:stock-logistic-warehouse

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 team Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-warehouse.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/stock-logistic-warehouse/7.0-port-base_product_merge into lp:stock-logistic-warehouse

2014-07-03 Thread Alexandre Fayolle - camptocamp
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 Stock and Logistic Core Editors is subscribed to branch 
lp:stock-logistic-warehouse.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community] github migration: proposale for helper for travis configuration

2014-07-02 Thread Alexandre Fayolle
Hello,

I've spent some time setting up
https://github.com/gurneyalex/maintainer-quality-tools

The goal is to provide a setup for running OCA Addons tests with travis
when using OpenERP 6.1 or 7.0 which don't have the odoo.py command. It
should also be possible in the near future to add support for 8.0, and
so as to keep a very similar .travis.yml file in the different branches.
Later this project could serve as host for more tools if needed.

An example addon using this is stock-logistic-warehouse (7.0 branch)
(which has a failed build, but also has no tests and lots of non pep8
compatible code... we will need to fix this)

If people agree, I would like to put this under the OCA umbrella, and
encourage its use for all addons projects. Maybe the porting
documentation could point to it. PR for improvements are welcome. :-)

Thanks for your attention.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~cedric-lebrouster/ocb-server/ocb-7.0-bug-1253052-parent-order into lp:ocb-server

2014-06-30 Thread Alexandre Fayolle - camptocamp
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 to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community] Merge pull request considered harmful

2014-06-27 Thread Alexandre Fayolle
Hello,

I found this blog post both interesting and useful :
http://blog.spreedly.com/2014/06/24/merge-pull-request-considered-harmful

Leonardo Pistone had shown me https://github.com/github/hub before and
it's a nice tool. And since odoo and the community addons are moving to
a new platform, I thought I'd share this piece of insight with all of you.

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix_red_test-afe into lp:account-budgeting

2014-06-26 Thread Alexandre Fayolle - camptocamp
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://bugs.launchpad.net/account-budgeting/+bug/1334605

For more details, see:
https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix_red_test-afe/+merge/224602

fix failing test

I think the failure is caused by a typo in the test source code. 
-- 
https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix_red_test-afe/+merge/224602
Your team Account Core Editors is requested to review the proposed merge of 
lp:~camptocamp/account-budgeting/7.0-fix_red_test-afe into lp:account-budgeting.
=== modified file 'budget/test/analytic_amount.yml'
--- budget/test/analytic_amount.yml	2014-04-23 13:53:39 +
+++ budget/test/analytic_amount.yml	2014-06-26 11:51:38 +
@@ -60,5 +60,5 @@
 line = self.browse(cr, uid, ref('budget_line_1'))
 assert abs(line.analytic_amount + 200.0)  1e-4, Analytic amount is not correct! -200.0 != %s % line.analytic_amount
 assert abs(line.analytic_real_amount + 12.0)  1e-4, Analytic real amount is not correct! -12.0 != %s % line.analytic_real_amount
-assert abs(line.analytic_diff_amount + 188.0)  1e-4, Analytic diff amount is not correct! 188.0 != %s % line.analytic_diff_amount
+assert abs(line.analytic_diff_amount - 188.0)  1e-4, Analytic diff amount is not correct! 188.0 != %s % line.analytic_diff_amount
 

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/account-closing/70-forecast-prepaid into lp:account-closing

2014-06-26 Thread Alexandre Fayolle - camptocamp
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). 



-- 
https://code.launchpad.net/~akretion-team/account-closing/70-forecast-prepaid/+merge/210066
Your team Account Core Editors is subscribed to branch lp:account-closing.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/account-consolidation/7.0-bug-1296740-elbati into lp:account-consolidation/7.0

2014-06-26 Thread Alexandre Fayolle - camptocamp
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 : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-add-reference-field-xls-mdh into lp:account-financial-report

2014-06-26 Thread Alexandre Fayolle - camptocamp
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 Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~luc-demeyer/account-financial-report/7.0-add-account_trial_balance_period_xls into lp:account-financial-report

2014-06-26 Thread Alexandre Fayolle - camptocamp
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. 


-- 
https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-add-account_trial_balance_period_xls/+merge/224024
Your team Account Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~luc-demeyer/account-financial-report/7.0-account_journal_report_xls-namespace-extra into lp:account-financial-report

2014-06-26 Thread Alexandre Fayolle - camptocamp
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 Report Core Editors is subscribed to branch 
lp:account-financial-report.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] OpenERP future and upgrades

2014-06-25 Thread Alexandre Fayolle
Hello,

Quick answers (but I'm not in Fabien's mind)

1. odoo is the new name of OpenERP.
2. odoo will keep working on premise
3. openerp / odoo v8 is *not* available (yet). Nothing has been
officially tagged as a release and the development branch is still very
much a moving target. There will be a migration solution for users of v7
provided by Odoo SA.

Alexandre

On 23/06/2014 19:13, Jonathan Vargas wrote:
 Hi there,

 Since new branding name Odoo appeared and OpenERP 8 is available too,
 I'd like to know a few things:

 1. Odoo and OpenERP will be the same piece of code on the same
 development place (https://github.com/odoo/odoo)?

 2. Will Odoo keep working the On Premise version available for
 download of will it be focused on the cloud?

 3. How current OpenERP 7 users could upgrade to latest stable version?
 Is Odoo 8 latest stable version?

 Thanks.


 -- 
 *
 *

 *Jonathan Vargas Rodríguez*
 CEO and Business Developent
 Alkaid · Open Source Business Software http://alkaid.cr
 *T:*4001 6259 Ext. 01 
 *M:*4001 6259 Ext. 51
 *G:*plus.google.com/+JonathanVargas
 https://plus.google.com/+JonathanVargas/
 *E:*jvar...@alkaid.cr mailto:jvar...@alkaid.cr



 ___
 Mailing list: https://launchpad.net/~openerp-community
 Post to : openerp-community@lists.launchpad.net
 Unsubscribe : https://launchpad.net/~openerp-community
 More help   : https://help.launchpad.net/ListHelp


-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~sebastien.beau/hr-timesheet/hr-timesheet-fix-project-id-not-required into lp:hr-timesheet

2014-06-24 Thread Alexandre Fayolle - camptocamp
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: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~serpentcs/server-env-tools/base_module_record into lp:server-env-tools

2014-06-23 Thread Alexandre Fayolle - camptocamp
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 maintain it, I'm 
willing to perform a detailed review of the code and help that person to 
improve the module. 
-- 
https://code.launchpad.net/~serpentcs/server-env-tools/base_module_record/+merge/196613
Your team Server Environment And Tools Core Editors is subscribed to branch 
lp:server-env-tools.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~alejandrosantana/account-financial-tools/7.0-account-financial_tools--add--account_account_extended_search into lp:account-financial-tools

2014-06-23 Thread Alexandre Fayolle - camptocamp
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 Reviewer/Maintainer is subscribed to branch 
lp:account-financial-tools.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~gdgellatly/server-env-tools/base-synchro-7.0 into lp:server-env-tools

2014-06-23 Thread Alexandre Fayolle - camptocamp
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 no good reason)

* there are no automated tests, and for functionality such as this one, this is 
definitely needed. 


-- 
https://code.launchpad.net/~gdgellatly/server-env-tools/base-synchro-7.0/+merge/200127
Your team Server Environment And Tools Core Editors is subscribed to branch 
lp:server-env-tools.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~pedro.baeza/account-financial-report/6.1-balance-reporting into lp:account-financial-report/6.1

2014-06-23 Thread Alexandre Fayolle - camptocamp
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 
lp:account-financial-report/6.1.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] 13 revisions removed from ocb-addons/7.0?

2014-06-20 Thread Alexandre Fayolle
On jeu. 19 juin 2014 10:36:08 CEST, Stefan wrote:
 On 18-06-14 16:40, Pedro Manuel Baeza Romero wrote:
 Done! Now, all OCB specific patches lost must be restored.


 Thanks Pedo, looking good!

 Stefan.


 ___
 Mailing list: https://launchpad.net/~openerp-community
 Post to : openerp-community@lists.launchpad.net
 Unsubscribe : https://launchpad.net/~openerp-community
 More help   : https://help.launchpad.net/ListHelp

Thanks to all for fixing this. I did not know about push --overwrite, 
and I'll stick to what I said at open days about bound branches... :-)

--
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/account-invoice-report/7_fix_1221570 into lp:account-invoice-report

2014-06-20 Thread Alexandre Fayolle - camptocamp
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: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~ajite/multi-company/multi-company-add-0001 into lp:multi-company

2014-06-20 Thread Alexandre Fayolle - camptocamp
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 
available in the addons path::

CRITICAL test_70 openerp.modules.module: Couldn't load module web
CRITICAL test_70 openerp.modules.module: No module named connector
ERROR test_70 openerp: Failed to initialize database `test_70`.
Traceback (most recent call last):
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/cli/server.py, line 
97, in preload_registry
db, registry = 
openerp.pooler.get_db_and_pool(dbname,update_module=update_module)
  File /home/afayolle/work/oerp/openobject-server/7.0/openerp/pooler.py, 
line 33, in get_db_and_pool
registry = RegistryManager.get(db_name, force_demo, status, 
update_module)
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/modules/registry.py, 
line 203, in get
update_module)
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/modules/registry.py, 
line 233, in new
openerp.modules.load_modules(registry.db, force_demo, status, 
update_module)
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/modules/loading.py, 
line 354, in load_modules
loaded_modules, update_module)
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/modules/loading.py, 
line 256, in load_marked_modules
loaded, processed = load_module_graph(cr, graph, progressdict, 
report=report, skip_modules=loaded_modules, perform_checks=perform_checks)
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/modules/loading.py, 
line 159, in load_module_graph
load_openerp_module(package.name)
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/modules/module.py, 
line 415, in load_openerp_module
getattr(sys.modules['openerp.addons.' + module_name], 
info['post_load'])()
  File /home/afayolle/work/oerp/web/7.0/addons/web/http.py, line 628, in 
wsgi_postload
openerp.wsgi.register_wsgi_handler(Root())
  File /home/afayolle/work/oerp/web/7.0/addons/web/http.py, line 517, in 
__init__
self.load_addons()
  File /home/afayolle/work/oerp/web/7.0/addons/web/http.py, line 580, in 
load_addons
m = __import__('openerp.addons.' + module)
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/modules/module.py, 
line 133, in load_module
mod = imp.load_module('openerp.addons.' + module_part, f, path, descr)
  File 
/home/afayolle/work/oca/multi-company/multi-company-add-0001/base_intercompany/__init__.py,
 line 23, in module
import connector
  File 
/home/afayolle/work/oca/multi-company/multi-company-add-0001/base_intercompany/connector.py,
 line 25, in module
from openerp.addons.connector.connector import Environment
  File 
/home/afayolle/work/oerp/openobject-server/7.0/openerp/modules/module.py, 
line 132, in load_module
f, path, descr = imp.find_module(module_part, ad_paths)
ImportError: No module named connector

2. The addon has tests (good!) but they trigger a number of ERROR level log
lines which may not be bugs, but I would at least need some confirmation of
this. Since the runbot considers the tests failed if there are such log
messages, this could be an issue for future automated tests:

TEST test_70 openerp.modules.module: module base_intercompany_sale: 
executing 1 `fast_suite` and/or `checks` sub-modules
TEST test_70 openerp.modules.module: test_01_creation 
(openerp.addons.base_intercompany_sale.tests.test_sale.test_sale_so2po)
ERROR test_70 openerp.osv.expression: The field 'Object' (model) can not be 
searched: non-stored function field without fnct_search
ERROR test_70 openerp.osv.expression: The field 'Object' (model) can not be 
searched: non-stored function field without fnct_search
TEST test_70 openerp.modules.module: test_02_creation_raise_exception 
(openerp.addons.base_intercompany_sale.tests.test_sale.test_sale_so2po)
ERROR test_70 openerp.osv.expression: The field 'Object' (model) can not be 
searched: non-stored function field without fnct_search
TEST test_70 openerp.modules.module: 
test_03_creation_without_write_permission 
(openerp.addons.base_intercompany_sale.tests.test_sale.test_sale_so2po)
ERROR test_70 openerp.osv.expression: The field 'Object' (model) can not be 
searched: non-stored function field without fnct_search
ERROR test_70 openerp.osv.expression: The field 'Object' (model) can not be 
searched: non-stored function field without fnct_search
TEST test_70 openerp.modules.module: test_01_creation 
(openerp.addons.base_intercompany_sale.tests.test_sale.test_sale_so2so)
ERROR test_70 openerp.osv.expression: The field 'Object' (model) can not be 
searched: non-stored 

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/multi-company/multi-company-action-user into lp:multi-company

2014-06-20 Thread Alexandre Fayolle - camptocamp
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 company to define a specific user to be used for the automatic actions. 
This field can
then be used by other modules defining such actions.

Even then, the concept of automatic action is not clear to me. An example 
would be welcome. 
-- 
https://code.launchpad.net/~akretion-team/multi-company/multi-company-action-user/+merge/179179
Your team Multi Company Core Editors is subscribed to branch lp:multi-company.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/ocb-addons/7.0-fix-regression-fill-inventory-OOM into lp:ocb-addons

2014-06-20 Thread Alexandre Fayolle - camptocamp
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:
https://code.launchpad.net/~numerigraphe-team/ocb-addons/7.0-fix-regression-fill-inventory-OOM/+merge/223928
-- 
https://code.launchpad.net/~numerigraphe-team/ocb-addons/7.0-fix-regression-fill-inventory-OOM/+merge/223928
Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~cedric-lebrouster/ocb-server/ocb-7.0-bug-1322191-db_maxconn into lp:ocb-server

2014-06-13 Thread Alexandre Fayolle - camptocamp
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 are not improving dead code here. 

Do you have a scenario at hand?
-- 
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 : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~cedric-lebrouster/ocb-server/ocb-7.0-bug-1322191-db_maxconn into lp:ocb-server

2014-06-13 Thread Alexandre Fayolle - camptocamp
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 : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe into lp:openerp-product-attributes

2014-06-11 Thread Alexandre Fayolle - camptocamp
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-product-attributes/7.0-add_product_gtin-afe/+merge/222760

replace product_gtin with chricar_product_gtin (from c2c-rd-addons), which 
works (whereas the existing module was not installable). 

I ported the history of changes of that addon using bazaar-extractor, and 
Alexis de Lattre did the actual porting to V7 + OCA coding standards
-- 
https://code.launchpad.net/~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe/+merge/222760
Your team Product Core Editors is requested to review the proposed merge of 
lp:~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe into 
lp:openerp-product-attributes.
=== added directory 'product_gtin'
=== removed directory 'product_gtin'
=== added file 'product_gtin/__init__.py'
--- product_gtin/__init__.py	1970-01-01 00:00:00 +
+++ product_gtin/__init__.py	2014-06-11 08:42:55 +
@@ -0,0 +1,26 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Product GTIN module for OpenERP
+#Copyright (c) 2004 TINY SPRL. (http://tiny.be) All Rights Reserved.
+#   Fabien Pinckaers f...@tiny.be
+#Copyright (c) 2008 ChriCar Beteilugungs- und Beratungs- GmbH
+#   All Rights Reserved.
+#   Ferdinand Gassauer t...@chricar.at
+#
+#This program is free software: you can redistribute it and/or modify
+#it under the terms of the GNU Affero General Public License as
+#published by the Free Software Foundation, either version 3 of the
+#License, or (at your option) any later version.
+#
+#This program is distributed in the hope that it will be useful,
+#but WITHOUT ANY WARRANTY; without even the implied warranty of
+#MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#GNU Affero General Public License for more details.
+#
+#You should have received a copy of the GNU Affero General Public License
+#along with this program.  If not, see http://www.gnu.org/licenses/.
+#
+##
+
+from . import product_gtin

=== removed file 'product_gtin/__init__.py'
--- product_gtin/__init__.py	2009-04-09 07:06:02 +
+++ product_gtin/__init__.py	1970-01-01 00:00:00 +
@@ -1,31 +0,0 @@
-##
-#
-# Copyright (c) 2004 TINY SPRL. (http://tiny.be) All Rights Reserved.
-#Fabien Pinckaers f...@tiny.be
-# Copyright (c) 2008 ChriCar Beteilugungs- und Beratungs- GmbH All Rights Reserved.
-#Ferdinand Gassauer t...@chricar.at
-#
-# WARNING: This program as such is intended to be used by professional
-# programmers who take the whole responsability of assessing all potential
-# consequences resulting from its eventual inadequacies and bugs
-# End users who are looking for a ready-to-use solution with commercial
-# garantees and support are strongly adviced to contract a Free Software
-# Service Company
-#
-# This program is Free Software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License
-# as published by the Free Software Foundation; either version 2
-# of the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
-#
-##
-
-import product_gtin
\ No newline at end of file

=== added file 'product_gtin/__openerp__.py'
--- product_gtin/__openerp__.py	1970-01-01 00:00:00 +
+++ product_gtin/__openerp__.py	2014-06-11 08:42:55 +
@@ -0,0 +1,39 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Product GTIN module for OpenERP
+#Copyright (C) 2004-2011 Tiny SPRL (http://tiny.be).
+#Copyright (C) 2010-2011 Camptocamp Austria (http://www.camptocamp.at)
+#
+#This program is free software: you can redistribute it and/or modify
+#it under the terms of the GNU Affero General Public License as
+#published by the Free Software Foundation, either version 3 of the
+#License, or (at your option) any later version.
+#
+#This program is distributed in the hope that it will be useful

[Openerp-community] git: branch names

2014-06-10 Thread Alexandre Fayolle
Hello,

I've noticed that git offers hierarchic branch names, and I think this
could be a useful tool for the management of branches and PR.

https://github.com/odoo-dev/odoo/branches seems to be using a flat
namespace similar to what was used in Launchpad, but I feel that this
could be vastly improved. Are there any github old timers who could
point us to the state of the art best practices here?

I've seen
http://stackoverflow.com/questions/273695/git-branch-naming-best-practices
and I like the username/feature/issue-tracker-number/short-description
scheme. Maybe a version number for the target branch should be added
somewhere.

Opinions?

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com


___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community] OpenERP launchpad bug list ?

2014-06-09 Thread Alexandre Fayolle
On 06/06/2014 11:28, Yann Papouin wrote:
 Hi everyone,

 After seeing Alexandre Fayolle starting to replicate LP bug reports
 into GitHub, I want to know where is the previous launchpad bugs
 reports list.


That was a mistake on my side. The intent was to replicate only the bug
reports for things fixed in OCB and not merged in Official. After
discussing this with Stefan Rijnhart and Raphael Valyi, we came to the
conclusion that just resubmitting the MP and PR should be enough and
that we would script this.

My appologies for the noise and inconvenience which my attempt caused.




 All projects are redirected from:
 https://bugs.launchpad.net/openobject-addons
 https://bugs.launchpad.net/openobject-server/
 https://bugs.launchpad.net/openerp-web/

 to:
 https://bugs.launchpad.net/bugs/bugtrackers/odoo-bugs
 *Bugs are tracked in Odoo bugtracker
 https://bugs.launchpad.net/bugs/bugtrackers/odoo-bugs. *

 Where no bug reports list can be found:
 To lookup old bugs you can search via Google or via
 https://bugs.launchpad.net/openobject;

 According to the transition wiki page:
 LP bugs stay on LP, but no new ones. New bugs = GitHub issues
 https://github.com/odoo/odoo/wiki/GitHub-Transition

 All my existing launchpad filters are not working anymore:

Painful I agree. Not sure if the Odoo team can do much about this
though. Migrating the bug base and MP base to github would have been
nice, but I'm not sure this will happen.


-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Bug 1070418] Re: budget position should support view accounts

2014-06-06 Thread Alexandre Fayolle - camptocamp
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 in OpenERP Community Backports (Addons):
  Fix Released
Status in OpenERP Community Backports (Addons) 6.1 series:
  Fix Released
Status in OpenERP Community Backports (Addons) 7.0 series:
  Fix Released
Status in Odoo Addons:
  Fix Committed

Bug description:
  Hello,

  In account_budget module, when creating a budget position, the user
  can select view accounts and also accounts with consolidation
  children, in addition to normal accounts.

  However, when viewing budgets with positions containing only view
  accounts, the practical amount field is always zero.

  Since these type of accounts are accepted as budget positions, the
  system should take into account children and consolidation children
  when computing the practical amount.

  How to reproduce:
  1. install account_budget module
  2. create some moves on a general account
  3. create a budget position containing one *view* account - debit and credit 
values are shown correctly in the budget position
  4. create a budget with one line referring to the above budget position - 
the practical amount is always 0

  I attach a merge proposal.

  -sbi

To manage notifications about this bug go to:
https://bugs.launchpad.net/ocb-addons/+bug/1070418/+subscriptions

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Bug 1108228] Re: action_stock_invoice_onshipping missing on stock.picking

2014-06-06 Thread Alexandre Fayolle - camptocamp
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 Backports (Addons):
  Fix Released
Status in OpenERP Community Backports (Addons) 7.0 series:
  Fix Released
Status in Odoo Addons:
  Fix Committed

Bug description:
  Invoicing miltiple lines in Deliveries to Invoice is not possible
  because of a missing action:

  Action:  missing on stock.picking
  Model: stock.picking

  This action exists on stock.picking.in and stock.picking.out but not
  on stock.picking.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ocb-addons/+bug/1108228/+subscriptions

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Bug 1146923] Re: General account should not be required on analytic entries

2014-06-06 Thread Alexandre Fayolle - camptocamp
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 Community Backports (Addons):
  Fix Released
Status in OpenERP Community Backports (Addons) 6.1 series:
  New
Status in OpenERP Community Backports (Addons) 7.0 series:
  Fix Released
Status in Odoo Addons:
  Fix Committed

Bug description:
  The manual for analytic accounts states[1] : Each analytic entry can
  be linked to a general account, or not, as you wish. Conversely, an
  entry in a general account can be linked to one, several, or no
  corresponding analytic accounts.

  But the field 'general_account_id' is defined as required=True in 
account/account_analytic_line.py
  I suggest that this requirement should be relaxed to allow greater 
flexibility in analytic accounting.

  [1] http://doc.openerp.com/v6.1/book/4/4_10_Accounts/index.html

  Lionel Sausin

To manage notifications about this bug go to:
https://bugs.launchpad.net/ocb-addons/+bug/1146923/+subscriptions

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Bug 1181291] Re: [Trunk/7.0] Refund payment: Error occurred while validating the field(s) amount_currency

2014-06-06 Thread Alexandre Fayolle - camptocamp
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) amount_currency

Status in OpenERP Community Backports (Addons):
  Fix Released
Status in OpenERP Community Backports (Addons) 7.0 series:
  Fix Released
Status in Odoo Addons:
  Fix Committed

Bug description:
  
http://7-0-11185.runbot.openerp.com/?db=7-0-11185-allts=1368805793379#id=62view_type=formmodel=account.invoicemenu_id=422action=462

  I create a purchase refund in USD

  I pay it with usd bank (USD journal)

  pay and

  ValidateError

  Error occurred while validating the field(s) amount_currency: The
  amount expressed in the secondary currency must be positive when the
  journal item is a debit and negative when if it is a credit.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ocb-addons/+bug/1181291/+subscriptions

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/sale-wkfl/product_procurement_type into lp:sale-wkfl

2014-06-04 Thread Alexandre Fayolle - camptocamp
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 lp:sale-wkfl.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Merge] lp:~camptocamp/sale-wkfl/product_procurement_type into lp:sale-wkfl

2014-06-04 Thread Alexandre Fayolle - camptocamp
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, test
  Lorenzo Battistini - Agile BG (elbati)
  Stefan Rijnhart (Therp) (stefan-therp)
  Leonardo Pistone - camptocamp (lpistone): code review
  Yannick Vaucher @ Camptocamp (yvaucher-c2c): code review, no tests

For more details, see:
https://code.launchpad.net/~camptocamp/sale-wkfl/product_procurement_type/+merge/222111

Here are two new addons who brings a simple field to manage several combinations
for two procurements fields : procure_method  supply_method :
First addon adds:
- Normal: MTS + buy
- Bill of Materials: MTO + produce
Second addon depends of sale_dropshipping addon and adds:
- Dropshipping: MTO + buy

I'm not sure if it's the right community branch to propose it, if you think 
about a better
place, let me know.

-- 
https://code.launchpad.net/~camptocamp/sale-wkfl/product_procurement_type/+merge/222111
Your team Sale Core Editors is subscribed to branch lp:sale-wkfl.
=== added directory 'product_procurement_type'
=== added file 'product_procurement_type/__init__.py'
--- product_procurement_type/__init__.py	1970-01-01 00:00:00 +
+++ product_procurement_type/__init__.py	2014-06-04 22:49:14 +
@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Author: Romain Deheele
+#Copyright 2014 Camptocamp SA
+#
+#This program is free software: you can redistribute it and/or modify
+#it under the terms of the GNU Affero General Public License as
+#published by the Free Software Foundation, either version 3 of the
+#License, or (at your option) any later version.
+#
+#This program is distributed in the hope that it will be useful,
+#but WITHOUT ANY WARRANTY; without even the implied warranty of
+#MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#GNU Affero General Public License for more details.
+#
+#You should have received a copy of the GNU Affero General Public License
+#along with this program.  If not, see http://www.gnu.org/licenses/.
+#
+##
+
+from . import product

=== added file 'product_procurement_type/__openerp__.py'
--- product_procurement_type/__openerp__.py	1970-01-01 00:00:00 +
+++ product_procurement_type/__openerp__.py	2014-06-04 22:49:14 +
@@ -0,0 +1,50 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Author: Romain Deheele
+#Copyright 2014 Camptocamp SA
+#
+#This program is free software: you can redistribute it and/or modify
+#it under the terms of the GNU Affero General Public License as
+#published by the Free Software Foundation, either version 3 of the
+#License, or (at your option) any later version.
+#
+#This program is distributed in the hope that it will be useful,
+#but WITHOUT ANY WARRANTY; without even the implied warranty of
+#MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#GNU Affero General Public License for more details.
+#
+#You should have received a copy of the GNU Affero General Public License
+#along with this program.  If not, see http://www.gnu.org/licenses/.
+#
+##
+
+{
+name: Product Procurement Types,
+version: 0.1,
+author: Camptocamp,
+license: AGPL-3,
+category: Sales,
+description:
+
+Add a procurement type selection on products.
+Let create products as :
+ - Standard
+ - Bill of Materials
+
+Choose Standard changes automatically
+procurement method as make to stock and supply method as buy.
+Choose Bill of Materials changes automatically
+procurement method as make to order and supply method as produce.
+
+,
+website: http://camptocamp.com;,
+depends: ['sale',
+'procurement',
+'mrp'],
+data: ['product_view.xml'],
+demo: [],
+test: ['test/test_onchange_procurement_type.yml'],
+active: False,
+installable: True,
+}

=== added file 'product_procurement_type/product.py'
--- product_procurement_type/product.py	1970-01-01 00:00:00 +
+++ product_procurement_type/product.py	2014-06-04 22:49:14 +
@@ -0,0 +1,139 @@
+# -*- coding: utf-8 -*-
+##
+#
+#Author: Romain Deheele
+#Copyright 2014 Camptocamp SA
+#
+#This program is free software: you can redistribute it and/or modify
+#it under the terms of the GNU Affero General Public License as
+#published by the Free Software Foundation, either version 3 of the
+#License

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-tools/7.0-fix-l10n_fr_siret-fix-lp1320918-rde into lp:account-financial-tools

2014-05-23 Thread Alexandre Fayolle - camptocamp
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 list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-tools/7.0-fix-l10n_fr_siret-fix-lp1320918-rde into lp:account-financial-tools

2014-05-23 Thread Alexandre Fayolle - camptocamp
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: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-tools/7.0-fix-l10n_fr_siret-fix-lp1320918-rde into lp:account-financial-tools

2014-05-19 Thread Alexandre Fayolle - camptocamp
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: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/7.0-fix-1319109 into lp:ocb-addons

2014-05-14 Thread Alexandre Fayolle - camptocamp
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
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/7.0-fix-1319095 into lp:ocb-addons

2014-05-14 Thread Alexandre Fayolle - camptocamp
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 
(https://docs.python.org/2.7/library/tempfile.html?highlight=namedtemporaryfile#tempfile.NamedTemporaryFile)


-- 
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1319095/+merge/219476
Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-05-12 Thread Alexandre Fayolle - camptocamp
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
-- 
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 list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-05-12 Thread Alexandre Fayolle - camptocamp
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 
lp:server-env-tools.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~invitu/ocb-server/7.0 into lp:ocb-server

2014-05-12 Thread Alexandre Fayolle - camptocamp
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 lp:ocb-server.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/rma-fix-lp1311509_rde into lp:openerp-rma

2014-05-06 Thread Alexandre Fayolle - camptocamp
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
-- 
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 : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/purchase-wkfl/7.0-bug-1312024-tafaru into lp:purchase-wkfl

2014-05-06 Thread Alexandre Fayolle - camptocamp
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 : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/hr-timesheet/70-fix-timesheet-task-bug1316456 into lp:hr-timesheet

2014-05-06 Thread Alexandre Fayolle - camptocamp
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: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into lp:purchase-wkfl

2014-05-06 Thread Alexandre Fayolle - camptocamp
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 : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/sale-wkfl/7.0-dropshipping-merge-po-lep into lp:sale-wkfl

2014-05-06 Thread Alexandre Fayolle - camptocamp
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 : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/ocb-7.0-fix_1302630_document_search_order_by-rde into lp:ocb-addons

2014-05-06 Thread Alexandre Fayolle - camptocamp
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: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


Re: [Openerp-community-reviewer] [Merge] lp:~cedric-lebrouster/ocb-server/ocb-7.0-bug-1253052-parent-order into lp:ocb-server

2014-04-25 Thread Alexandre Fayolle - camptocamp
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 openerp.tools.yaml_import: AssertionError in Python code : 
After duplication, previous record must have old childs records only
TEST test_ocb70 openerp.tools.yaml_import: 3. Duplicate the children then 
reassign them to the new parent (1st method) and check the parent_store 
structure.
ERROR test_ocb70 openerp.tools.yaml_import: AssertionError in Python code : 
After duplication, previous record must have old childs records only
TEST test_ocb70 openerp.tools.yaml_import: 4. Duplicate the children then 
reassign them to the new parent (2nd method) and check the parent_store 
structure.
ERROR test_ocb70 openerp.tools.yaml_import: AssertionError in Python code : 
After duplication, previous record must have old childs records only
TEST test_ocb70 openerp.tools.yaml_import: 5. Duplicate the children then 
reassign them to the new parent (3rd method) and make sure the parent_store 
structure is still right.
ERROR test_ocb70 openerp.tools.yaml_import: AssertionError in Python code : 
After duplication, previous record must have old childs records only

Can you please check this, and adapt the tests (which seem to have some 
assumptions broken by your changes) or fix the code in yaml_import?
-- 
https://code.launchpad.net/~cedric-lebrouster/ocb-server/ocb-7.0-bug-1253052-parent-order/+merge/209708
Your team OpenERP Community Backports Team is subscribed to branch 
lp:ocb-server.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


[Openerp-community-reviewer] [Merge] lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into lp:purchase-wkfl

2014-04-25 Thread Alexandre Fayolle - camptocamp
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
-- 
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 : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp


  1   2   3   4   >