[Openerp-community] [ANN] OCA code sprint in Louvain la Neuve on June 1-2, 2015
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
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
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
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
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
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
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...
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...
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
@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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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?
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
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
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
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
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
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
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
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
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 ?
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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