Avoid the # box, I find it very irking to have that many heavy
characters like that.
Besides: E265 block comment should start with '# ' (single space after
the first hash).
How about:
# -*- coding: utf-8 -*-
#
# License, author and contributors information in:
# __openerp__.py file at the root
I agree.
In Object Oriented Programming, an object is an instance of a class, not
the class itself.
Since these are relational models, _model makes more sense to me.
I also like _pool as it is clear that it comes from self.pool (in v7).
There's also _registry or _reg.
Anyway, _obj doesn't make
Hello community,
I am writing for two reasons. One is to inform you that there has been a
rather drastic change to what Travis tests in code quality. Second is a
proposal for a more sensible approach to changes to automatic tests.
Firstly, there are new checks done by pylint [1]:
(all
Edit: Proposal has been added to the wiki page
https://github.com/OCA/maintainer-quality-tools/wiki
Feedback is appreciated.
Le 2014-11-13 11:42, Sandy Carter a écrit :
Hello community,
I am writing for two reasons. One is to inform you that there has been a
rather drastic change to what
these modules are fixed.
[1] https://github.com/OCA/maintainer-quality-tools/pull/113
Le 2014-11-13 11:42, Sandy Carter a écrit :
Hello community,
I am writing for two reasons. One is to inform you that there has been a
rather drastic change to what Travis tests in code quality. Second is a
proposal
.
Please raise your objection on the merge proposal.
Le 2014-11-13 12:24, Georges Racinet a écrit :
On 11/13/2014 06:21 PM, Georges Racinet wrote:
Hi Sandy,
On 11/13/2014 06:13 PM, Sandy Carter wrote:
Following my proposed procedure, I am informing the community that we
intend to add a new style
IMHO I like more use a .md file into same project. (not wiki)
Wiki is in .md file format.
It is better for this highlevel explanation to be in the wiki, IMHO
You can add a new feature in PR and add new doc in same PR.
md file is more portable.
*For your fork
*For you use in other git
Le 2014-10-22 04:09, Leonardo Pistone a écrit :
I think we should wipe the document, and start entering the
conventions that are already accepted, that is those that we already
check with flake8 and pylint.
+1
On the topic of one-file-per-class, I understand the reasons, but I
vote -1
Hi,
My issue with leaving it to subjective criticism is:
Firstly, if you're new and learning the ropes to contribution to OCA and
you do your homework, you're going to do two things to make sure your
module gets integrated. First you'll check existing modules and emulate
what they do and second
Hi,
I would like propose adding the following to the Module section in
http://odoo-community.org/page/website.how-to
Each XML and python script should be separated and named by model.
example:
_name = 'sale.order' # goes in a file named sale_order.py and the class
name should be SaleOrder
It's been done already:
https://github.com/nbessi/odoo_new_api_guideline
Le 2014-08-12 16:45, David Arnold - El Alemán a écrit :
+1 for test coverage
1. I anyone willing to write (in his limited time) a tutorial to be
understodd by everyone?
2. We could maka a general call to test cases as
Le 2014-08-12 05:21, Alexandre Fayolle a écrit :
IMO it is more important to improve test coverage than to port to the
new API.
Absolutely, increasing coverage is the best way to avoid regressions
when reimplementing, such as coverting from one API to another.
signature.asc
Description:
Review: Resubmit
Please move this MP to https://github.com/OCA/l10n-canada
More info at
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
https://code.launchpad.net/~openerp-community/openerp-canada/7.0-missing-payroll-access/+merge/226187
Your team
Review: Resubmit
This PR has been moved to https://github.com/OCA/connector-sage-50/pull/7
Please continue further development there.
--
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50/+merge/191893
Your team OpenERP Community Reviewer/Maintainer is
The proposal to merge
lp:~savoirfairelinux-openerp/knowledge-addons/document_multiple_records into
lp:knowledge-addons/7.0 has been updated.
Status: Approved = Rejected
For more details, see:
Review: Resubmit
This MP have been moved to MP to https://github.com/OCA/knowledge/pull/5
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/document_multiple_records/+merge/206960
Your team OpenERP Community Reviewer/Maintainer is requested to review the
proposed merge
Review: Resubmit
This MP have been moved to MP to https://github.com/OCA/knowledge/pull/6
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:knowledge-addons/7.0.
--
Mailing
The proposal to merge lp:~savoirfairelinux-openerp/knowledge-addons/cmis into
lp:knowledge-addons/7.0 has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258
--
Review: Resubmit
This MP have been moved to MP to https://github.com/OCA/knowledge/pull/7
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
Your team OpenERP Community Reviewer/Maintainer is requested to review the
proposed merge of
The proposal to merge lp:~savoirfairelinux-openerp/knowledge-addons/cmis_read
into lp:knowledge-addons/7.0 has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
--
The proposal to merge lp:~savoirfairelinux-openerp/knowledge-addons/cmis_write
into lp:knowledge-addons/7.0 has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_write/+merge/213940
--
This is for cases where one partner (Mike Fletcher President of Agrolait) can
be part of another company (Mike Fletcher VP of AsusTEK) in a different
function.
The module helps track these functions.
--
Ported to https://github.com/OCA/hr/pull/3
--
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/hr_employee_firstname/+merge/204065
Your team HR Core Editors is requested to review the proposed merge of
lp:~savoirfairelinux-openerp/openerp-hr/hr_employee_firstname into
The proposal to merge
lp:~savoirfairelinux-openerp/openerp-hr/hr_employee_firstname into
lp:openerp-hr has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/hr_employee_firstname/+merge/204065
--
Review: Approve
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/document_multiple_records/+merge/206960
Your team OpenERP Community Reviewer/Maintainer is requested to review the
proposed merge of
lp:~savoirfairelinux-openerp/knowledge-addons/document_multiple_records
The proposal to merge
lp:~savoirfairelinux-openerp/knowledge-addons/document_multiple_records into
lp:knowledge-addons/7.0 has been updated.
Status: Needs review = Approved
For more details, see:
+==
+
+This module allows you to add firstname and lastname in employee form,
+and concatenate both in name field.
+
+Contributors
+
+* El Hadji Dem (elhadji@savoirfairelinux.com)
+* Sandy Carter (sandy.car...@savoirfairelinux.com)
+,
+'depends': [
+'hr
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Le 2014-06-26 10:50, Guewen Baconnier a écrit :
4. Sandy added this great stuff to the project.
Thank you :)
5. Set modules to uninstallable with:
ack installable --py -l | xargs sed s/[\|']installable[\|']:
True/'installable': False/ -i
Diff comments:
=== added directory 'cmis_write'
=== added file 'cmis_write/__init__.py'
--- cmis_write/__init__.py1970-01-01 00:00:00 +
+++ cmis_write/__init__.py2014-05-29 22:13:35 +
@@ -0,0 +1,26 @@
+# -*- encoding: utf-8 -*-
It's a little incontinent, but the setting to skip tests if the .travis.yml
file is missing is off by default.
To enable this setting, if you have admin rights to the repos,
Log into https://travis-ci.org using your github credentials.
Go to https://travis-ci.org/profile/OCA and click on the
On the topic of Runbot.
Here at SLF, I have managed to deploy it for our own uses.
I am sure many of you have read it already but for those who didn't here's the
result of my experiment[1].
Since then, I have made an addon to runbot which gives more customisability to
what runbot runs, mainly
to have that integrated any time soon.
--
Sandy
- Mail original -
De: David Arnold - El Alemán da...@elaleman.co
À: Joël Grand-Guillaume joel.grandguilla...@camptocamp.com
Cc: Sandy Carter sandy.car...@savoirfairelinux.com, openerp-community
openerp-community@lists.launchpad.net
Envoyé
I do not have the rights to add repos to OCA, but runbot-addons has a nice ring
to it ;)
--
Sandy
- Mail original -
De: Joël Grand-Guillaume joel.grandguilla...@camptocamp.com
À: Sandy Carter sandy.car...@savoirfairelinux.com
Cc: Moises Lopez moylop...@vauxoo.com, openerp-community
Thank you for your review, Guewen, I learned a few things.
Diff comments:
=== added directory 'cmis_read'
=== added file 'cmis_read/__init__.py'
--- cmis_read/__init__.py 1970-01-01 00:00:00 +
+++ cmis_read/__init__.py 2014-06-12 19:57:17 +
@@ -0,0 +1,25 @@
+# -*- encoding:
Looking a lot better, still issues about the comments preceding functions which
would be more useful as docstrings.
Diff comments:
=== added directory 'cmis'
=== added file 'cmis/__init__.py'
--- cmis/__init__.py 1970-01-01 00:00:00 +
+++ cmis/__init__.py 2014-06-12 19:47:23 +
@@
Review: Approve
Looking better
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:knowledge-addons/7.0.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
You have been requested to review the proposed merge of
lp:~savoirfairelinux-openerp/knowledge-addons/cmis_read into
lp:knowledge-addons/7.0.
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
Add cmis_read; It allows to use the
Review: Needs Fixing
Diff comments:
=== modified file 'l10n_ca_hr_payroll/__openerp__.py'
--- l10n_ca_hr_payroll/__openerp__.py 2013-02-27 20:02:47 +
+++ l10n_ca_hr_payroll/__openerp__.py 2014-06-11 15:00:16 +
@@ -38,6 +38,7 @@
'l10n_ca_toponyms',
],
'data': [
Slashes in git branch names can cause problems with some helper tools as
it messes up the .git/refs/ directory.
The username and feature/fix/i18n part seems redundant to me, these
belong in commit messages and PR descriptions.
The issue tracker number can be put in the commit message: i.e. Fixes
You can find the runbot source at on odoo-extra[1].
Documentation is supposed to come, but I am sure you can ask around there.
[1] https://github.com/odoo/odoo-extra
--
Sandy
- Mail original -
De: Ovnicraft ovnicr...@gmail.com
À: openerp-community@lists.launchpad.net
Envoyé: Mardi 10
An update:
I have commited a project to github: lp:openerp-travel to play around
with travis-ci.org and coverall.io
A .travis.yml[1], a .coveragerc[2] file and a README.md[3] with travis
and coveralls badge have been added to the project. This should serve as
a template (or an example) for the
Hello Community.
I have a proposal for submit before we start porting our projects to
8.0.
I suggest we add travis.yml to the root of each addon repository.
That way, we can use the free service of travis-ci.org to do initial
testing of each MP and each commit.
This will greatly reduce the
Review: Needs Fixing
For better usability and security, your sanitize function should wrap the query
function, the same way OE does, so that there is no way to call the query the
wrong way.
Something along the lines of
safe_query( SELECT cmis:name, cmis:createdBy, cmis:objectId,
Review: Needs Information
l.225,259 can you please explain what you do here. It seems you attempt to
write to a directory to test if you have write permission, surely there are
safer ways of doing this.
In you unittest, can you test more of your functions, I know there is a
challenge of not
Review: Needs Fixing
l.469,502: Queries are still not fully sanitized, any quotes or percent sizes
in the input will result in unexpected behaviour. This is a security risk and a
major bug potential.
I highly recommend you to add a function to sanitize input in the cmis module
for queries and
@Stefan, yes that would work, we could even be backwards compatible and have a
symlink
openerp_nag - openerp_nag.py
--
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/nag_refactor/+merge/214445
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
Review: Needs Fixing code review, no test
l.288 looks like you forgot to put anything in for group_id:id
purchase_profit_forecast/__openerp__.py:
Manifest license key is missing
Deprecated Manifest key ('active')
Deprecated Manifest key ('certificate')
Flake8:
report/analytic.py:3:1: E302
Review: Needs Fixing code review, pep8
l.112,464,2878 You can remove these empty files
Flake8
delivery_editable_dates/__openerp__.py:22:1: O603 Manifest license key is
missing
delivery_editable_dates/__openerp__.py:28:5: O600 Warning unknown Manifest key
('caterogy')
Review: Needs Fixing
Flake8:
cmis/connector.py:23:1: F401 'fields' imported but unused
cmis/connector.py:23:1: F401 'orm' imported but unused
l.270 you still need to escape literals
http://wiki.alfresco.com/wiki/CMIS_Query_Language#Literals
Review: Needs Fixing
l.416 Missing context propagation
Spelling:
You wrote OpenErp a few times. This should be OpenERP.
The description is oddly formulated, you may want it proofread.
Flake8:
cmis_write/__openerp__.py:58:26: W291 trailing whitespace
cmis_write/ir_attachment.py:125:5: E265 block
Still no unittests
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_write/+merge/213940
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:knowledge-addons/7.0.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Review: Needs Fixing
l.471 Still severe bug and injection potential
Try:
filename = sql%' OR '1' = '1' OR '%injection
CMIS must provide a code escape function, otherwise use OpenERP's. It is
important that you don't do this manually.
https://en.wikipedia.org/wiki/Sql_injection
There are also
Review: Needs Fixing
except_osv is depricated, please use except_orm
l.213 missing context propagation
l.276 POSSIBLE SQL INJECTION with filenames which contain quotes, please use
appropriate code escapes from the library (don't write your own)
No Unittests
Flake8:
cmis/cmis_model.py:25:1: F401
Review: Needs Fixing
This MP should have a prerequisite branch:
lp:~savoirfairelinux-openerp/knowledge-addons/cmis
l.301 missing context propagation
No unittests
Flake8
cmis_write/ir_attachment.py:73:45: W291 trailing whitespace
cmis_write/ir_attachment.py:74:53: W291 trailing whitespace
Review: Needs Fixing
Missing security model for cmis.backend
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:knowledge-addons/7.0.
--
Mailing list:
Review: Needs Fixing
l.377 osv.TransientModel should be orm.TransientModel
l.504 POSSIBLE SQL INJECTION: (with quotes in a filename) please use the
library's code escapes, don't write your own
Flake8:
cmis_read/wizard/document_wizard.py:102:9: F841 local variable 'file_name' is
assigned to but
Review: Needs Fixing
openerp.modules.loading: The transient model ir.attachment.dms
(ir.attachment.dms) should not have explicit access rules!
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
Your team OpenERP Community Reviewer/Maintainer is
Injection is still possible. It is an entry point for malicious queries or
accidental breaks.
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:knowledge-addons/7.0.
--
+1 we can always use active contributors as reviewers!
--
Sandy
- Original Message -
From: Stefan Rijnhart ste...@therp.nl
To: openerp-community-revie...@lists.launchpad.net,
openerp-community@lists.launchpad.net
Sent: Thursday, May 1, 2014 5:17:58 AM
Subject: Re:
+1 we can always use active contributors as reviewers!
--
Sandy
- Original Message -
From: Stefan Rijnhart ste...@therp.nl
To: openerp-community-reviewer@lists.launchpad.net,
openerp-commun...@lists.launchpad.net
Sent: Thursday, May 1, 2014 5:17:58 AM
Subject: Re:
Sorry. I didn't explain very well.
I have MPs for OCB, 7.0 and trunk for this bug.
OE trunk doesn't have both MP (server and web) because it too much, therefore
web doesn't have an MP.
As for server, commits up to r5297 ([FIX] Fixed instance of ValidateError not
being translated) have been
Review: Needs Fixing pep8
crm_firstname/__openerp__.py:22:1: O603 Manifest license key is missing
crm_firstname/__openerp__.py:35:19: E231 missing whitespace after ','
crm_firstname/__openerp__.py:37:11: E126 continuation line over-indented for
hanging indent
crm_firstname/__openerp__.py:39:11:
It would make the tests fail, but this problem is widespread in OE.
For example test_20_message_post
(openerp.addons.mail.tests.test_mail_features.test_mail) fails when a
translator changes the value. It just doesn't fail in en_US because the values
haven't changed. The same is not true for
I have already made some MPs for upstream, but the code is pretty different.
--
https://code.launchpad.net/~savoirfairelinux-openerp/ocb-server/translate_warnings_1297525-trunk/+merge/212736
Your team OpenERP Community Reviewer/Maintainer is requested to review the
proposed merge of
Review: Approve
LGTM, thanks
The future module is worth looking at for future.str_as_unicode
https://pypi.python.org/pypi/future/0.3.1
--
https://code.launchpad.net/~openerp-community/openerp-canada/7.0-l10n_ca_account_check_writing/+merge/216708
Your team OpenERP Community is subscribed to
Review: Needs Information
@Stefan they give a satisfactory message when translated (no update to .po
needed)
I don't want to change too much from the original code, but if you feel it's
really needed, I can replace them with your suggestions.
--
You have been requested to review the proposed merge of
lp:~savoirfairelinux-openerp/ocb-web/translate_warnings_1297525 into lp:ocb-web.
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/ocb-web/translate_warnings_1297525/+merge/212739
Allows the title of warning boxes
You have been requested to review the proposed merge of
lp:~savoirfairelinux-openerp/ocb-server/translate_warnings_1297525-trunk into
lp:ocb-server.
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/ocb-server/translate_warnings_1297525-trunk/+merge/212736
When
Review: Abstain
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_modules/+merge/217775
Your team Partner and Contact Core Editors is requested to review the proposed
merge of
You have been requested to review the proposed merge of
lp:~savoirfairelinux-openerp/knowledge-addons/cmis_write into
lp:knowledge-addons/7.0.
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_write/+merge/213940
Add cmis_write: It allows to
Done.
Please make sure this doesn't have funny side effects and translates properly.
--
https://code.launchpad.net/~savoirfairelinux-openerp/ocb-server/translate_warnings_1297525-trunk/+merge/212736
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-server.
--
Mailing
The tests fail anyway in any other language, especially in yml tests
Should I fix that assert too.
--
https://code.launchpad.net/~savoirfairelinux-openerp/ocb-server/translate_warnings_1297525-trunk/+merge/212736
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-server.
Basically, if I change the labels, I would change the message in the tests.
Ideally, an assertRaises should be used there, but I have no idea if yml tests
support that.
--
https://code.launchpad.net/~savoirfairelinux-openerp/ocb-server/translate_warnings_1297525-trunk/+merge/212736
Your team
Review: Approve
--
https://code.launchpad.net/~camptocamp/lp-community-utils/nag-nameerror-1312682/+merge/217216
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:lp-community-utils.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
You have been requested to review the proposed merge of
lp:~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact
into lp:partner-contact-management.
For more details, see:
Review: Abstain
@Holger, took your comments into consideration
Taking myself out of the Reviewer list since I have contributed.
--
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/document_multiple_records/+merge/206960
Your team OpenERP Community Reviewer/Maintainer is
You have been requested to review the proposed merge of
lp:~savoirfairelinux-openerp/knowledge-addons/document_multiple_records into
lp:knowledge-addons/7.0.
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/document_multiple_records/+merge/206960
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Thank you for communicating this.
Py2 is being kept on life support, but all the new features are in Py3.
We should all be thinking about forward compatibility to 3.
I recommend checking out By Your Bootstraps: Porting Your Application
to Python3
Blueprint changed by Sandy Carter (http://www.savoirfairelinux.com):
Whiteboard changed:
Gather information about an OpenERP addon MP
* Has this MP been reviewed before by an automated script?
- * Does it contain message identifying as automated?
- * Has there been a new commit since last
Review: Needs Fixing
@Yannick,
Thank you for pointing that out. Indeed, maiden name might be a charged,
non-gender neutral name for the field and should be changed.
--
Review: Approve
l.120 is CamelCase the standard now?
--
https://code.launchpad.net/~camptocamp/sale-wkfl/add-jit-module-for-service-product-jge/+merge/214719
Your team Sale Core Editors is subscribed to branch lp:sale-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
@Yannick Good to know, thank you.
--
https://code.launchpad.net/~camptocamp/sale-wkfl/add-jit-module-for-service-product-jge/+merge/214719
Your team Sale Core Editors is subscribed to branch lp:sale-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Sandy Carter (http://www.savoirfairelinux.com) has proposed merging
lp:~savoirfairelinux-openerp/lp-community-utils/nag_refactor into
lp:lp-community-utils.
Requested reviews:
OpenERP Community Reviewer/Maintainer (openerp-community-reviewer)
Guewen Baconnier @ Camptocamp (gbaconnier-c2c
You have been requested to review the proposed merge of
lp:~savoirfairelinux-openerp/lp-community-utils/nag_refactor into
lp:lp-community-utils.
For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/nag_refactor/+merge/214445
Moved functions used in
Blueprint changed by Sandy Carter (http://www.savoirfairelinux.com):
Whiteboard set to:
Gather information about an OpenERP addon MP
* Has this MP been reviewed before by an automated script?
* Does it contain message identifying as automated?
* Has there been a new commit since last
Blueprint changed by Sandy Carter (http://www.savoirfairelinux.com):
Whiteboard changed:
Gather information about an OpenERP addon MP
* Has this MP been reviewed before by an automated script?
* Does it contain message identifying as automated?
* Has there been a new commit since last
Two commits have been added:
r5296 from OpenERP's support
r5297 from our team for a case I missed of error message translation
--
https://code.launchpad.net/~savoirfairelinux-openerp/ocb-server/translate_warnings_1297525-trunk/+merge/212736
Your team OpenERP Community Backports Team is subscribed
The proposal to merge
lp:~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_partner_member_observer
into lp:partner-contact-management has been updated.
Status: Needs review = Rejected
For more details, see:
@Guewen, thank you for the link, I was wondering if such a tool existed.
--
https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/branch_pep8/+merge/205260
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:lp-community-utils.
--
Mailing list:
Review: Approve
--
https://code.launchpad.net/~therp-nl/server-env-tools/7.0-auth_from_http_basic/+merge/202316
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 :
@yanina-aular thank you for fixing that, please backport that commit here
--
https://code.launchpad.net/~camptocamp/ocb-addons/ocb-7.0-project_fix_doc_count-lp1251773-vauxoo+afe/+merge/209010
Your team OpenERP Community Backports Team is subscribed to branch
lp:ocb-addons.
--
Mailing list:
Sandy Carter (http://www.savoirfairelinux.com) has proposed merging
lp:~savoirfairelinux-openerp/ocb-server/translate_warnings_1297525-trunk into
lp:ocb-server.
Requested reviews:
OpenERP Community Backports Team (ocb)
Related bugs:
Bug #1297525 in OpenERP Web: Title of except_orm Warning
(['bzr', 'pull', '-d', self.target_dir],
env=SUBPROCESS_ENV)
It would be helpful to either have the option to set the --overwrite
flag, or to have it one by default.
What do you think?
--
Sandy Carter
Consultant en logiciel libre
Développeur OpenERP
Savoir-faire
Sandy Carter (http://www.savoirfairelinux.com) has proposed merging
lp:~savoirfairelinux-openerp/openerp-hr/department_sequence_concatination_NoneType
into lp:openerp-hr.
Requested reviews:
HR Core Editors (hr-core-editors)
For more details, see:
https://code.launchpad.net/~savoirfairelinux
For one, I do not use OCB, so there would be no point for me to propose a merge
into it if I want to use it. ;)
Second, I made a mistake in my comment when saying OCB, I meant
OpenObject-Addons.
You are absolutely right about the authorship, I will fix that. Thanks for
pointing that out with a
@Guewen
I reconstructed the history to the best of my abilities.
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact/+merge/203979
Your team Partner and Contact Core Editors is subscribed to branch
Community, any update on this?
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_partner_member_observer/+merge/204041
Your team Partner and Contact Core Editors is requested to review the proposed
merge of
Community, any update on this?
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_res_bank_account/+merge/204047
Your team Partner and Contact Core Editors is requested to review the proposed
merge of
Review: Approve
--
https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/partner-contact-management-base_contact_add_modules/+merge/204016
Your team Partner and Contact Core Editors is requested to review the proposed
merge of
Review: Needs Fixing code review, pep8
Looks, good, just a few pep8 issues:
openerp-nag:72:9: E126 continuation line over-indented for hanging indent
MY_READY: My MPs ready to land,
^
openerp-nag:73:9: E126 continuation line over-indented for hanging indent
MY_PENDING: My
1 - 100 of 236 matches
Mail list logo