This one is set as Superseeded. Seems fine. No change needed.
--
https://code.launchpad.net/~laetitia-gangloff/hr-timesheet/hr_attendance_analysis_externalise_contract_calendar/+merge/182565
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
The proposal to merge lp:~sylvain-legal/server-env-tools/7.0-flake8 into
lp:server-env-tools has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-flake8/+merge/225456
--
Completely agree that's why you always have my approval.
--
https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-flake8/+merge/225456
Your team Server Environment And Tools Core Editors is subscribed to branch
lp:server-env-tools.
--
Mailing list:
Review: Approve code review, no test
LGTM
--
https://code.launchpad.net/~therp-nl/server-env-tools/6.1-web_base_url_freeze/+merge/225651
Your team Server Environment And Tools Core Editors is subscribed to branch
lp:server-env-tools/6.1.
--
Mailing list:
The proposal to merge
lp:~akretion-team/server-env-tools/module-configuration-helper-dbl into
lp:server-env-tools has been updated.
Status: Needs review = Approved
For more details, see:
Review: Resubmit
Hello David,
I'm ok to merge this, I even tried to merge it before this project goes on
Github. However, it seems you created that module on an empty branch.
Could you redo/rebase this?
Might be better to do it directly in git ?
--
The proposal to merge
lp:~akretion-team/server-env-tools/module-configuration-helper-dbl into
lp:server-env-tools has been updated.
Status: Approved = Needs review
For more details, see:
Review: Approve code review, no test
l.330 you need to be sure context is a dict before using get
l.536 you can use dict setter
dict['split_key'] = [(3, id) for id
in vals.get(split_key, False)[0][2]]
Thaught this wasn't the target here.
I'll approve already as we want
s/Thaught/Though
--
https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-flake8/+merge/225456
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 :
Review: Resubmit pr created on github
--
https://code.launchpad.net/~akretion-team/server-env-tools/server-env-tools/+merge/91
Your team Server Environment And Tools Core Editors is subscribed to branch
lp:server-env-tools.
--
Mailing list:
Review: Resubmit moved on github
This project is now hosted on https://github.com/OCA/sale-reports. Please move
your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
Review: Needs Fixing
LGTM appart for the variable _ though we are not using translation here.
--
https://code.launchpad.net/~numerigraphe-team/report-print-send/7.0-no-lock-in-update-1308635-ls/+merge/216353
Your team Report Printing and Sending Core Editors is subscribed to branch
Review: Approve code review, no test
Thanks for the changes
LGTM
--
https://code.launchpad.net/~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls/+merge/219787
Your team Report Printing and Sending Core Editors is subscribed to branch
lp:report-print-send.
--
Mailing list:
The proposal to merge
lp:~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls into
lp:report-print-send has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls/+merge/219787
--
Review: Resubmit moved on github
This project is now hosted on https://github.com/OCA/report-print-send. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
Review: Resubmit moved on github
This project is now hosted on https://github.com/OCA/report-print-send. Please
move your proposal there. This guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
Thanks Pedro
Change made
--
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1335887/+merge/225028
Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Review: Approve code review, no test
Fixed mark down in __openerp__.py and improved it a bit.
LGTM
--
https://code.launchpad.net/~camptocamp/sale-reports/vre-sale_order_proformat/+merge/223563
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:sale-reports.
--
Mailing
The proposal to merge lp:~camptocamp/sale-reports/vre-sale_order_proformat into
lp:sale-reports has been updated.
Status: Merged = Approved
For more details, see:
https://code.launchpad.net/~camptocamp/sale-reports/vre-sale_order_proformat/+merge/223563
--
The proposal to merge lp:~camptocamp/sale-reports/vre-sale_order_proformat into
lp:sale-reports has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~camptocamp/sale-reports/vre-sale_order_proformat/+merge/223563
--
The proposal to merge lp:~camptocamp/sale-reports/vre-sale_order_proformat into
lp:sale-reports has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~camptocamp/sale-reports/vre-sale_order_proformat/+merge/223563
--
Review: Approve code review, no test
LGTM as
https://code.launchpad.net/~initos.com/account-invoice-report/7.0-fix_lang_for_draft/+merge/224109
--
https://code.launchpad.net/~initos.com/sale-reports/7.0-fix_lang_for_draft/+merge/224106
Your team OpenERP Community Reviewer/Maintainer is
The proposal to merge lp:~initos.com/sale-reports/7.0-fix_lang_for_draft into
lp:sale-reports has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~initos.com/sale-reports/7.0-fix_lang_for_draft/+merge/224106
--
This project is now hosted on https://github.com/OCA/carriers-deliveries.
Please move your proposal there if you still want to merge it once fixed. This
guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
This project is now hosted on https://github.com/OCA/carriers-deliveries.
Please move your proposal there if you still want to merge it once fixed. This
guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
(sorry David should have merged it
This project is now hosted on https://github.com/OCA/carriers-deliveries.
Please move your proposal there if you still want to merge it once fixed. This
guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
This project is now hosted on https://github.com/OCA/carriers-deliveries.
Please move your proposal there if you still want to merge it once fixed. This
guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
Note to myself
This project is now hosted on https://github.com/OCA/carriers-deliveries.
Please move your proposal there if you still want to merge it once fixed. This
guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
--
The proposal to merge
lp:~camptocamp/account-invoice-report/vre_account_invoice_report into
lp:account-invoice-report has been updated.
Status: Needs review = Rejected
For more details, see:
Review: Approve code review, no test
LGTM
--
https://code.launchpad.net/~initos.com/account-invoice-report/7.0-fix_lang_for_draft/+merge/224109
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-invoice-report.
--
Mailing list:
The proposal to merge
lp:~initos.com/account-invoice-report/7.0-fix_lang_for_draft into
lp:account-invoice-report has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~initos.com/account-invoice-report/7.0-fix_lang_for_draft/+merge/224109
--
Thanks for the fix.
--
https://code.launchpad.net/~initos.com/account-invoice-report/7.0-fix_lang_for_draft/+merge/224109
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-invoice-report.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post
Replay isn't working yet and passing from lp to github will make us having to
replay all the MP we did.
I think that while we don't have OCB fully working on github we can fill the
gap by merging that kind of MP in OCB on launchpad.
--
The proposal to merge
lp:~acsone-openerp/account-invoice-report/invoice_webkit_fr_translation into
lp:account-invoice-report has been updated.
Status: Needs review = Approved
For more details, see:
I merge this before migration to github. Missing 1 approve but only touch
translation so this is not critical.
--
https://code.launchpad.net/~acsone-openerp/account-invoice-report/invoice_webkit_fr_translation/+merge/214179
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
Thanks Leatitia for this MP
--
https://code.launchpad.net/~acsone-openerp/account-invoice-report/invoice_webkit_fr_translation/+merge/214179
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-invoice-report.
--
Mailing list:
The proposal to merge
lp:~therp-nl/account-invoice-report/account-invoice-report-7.0_delivery_address
into lp:account-invoice-report has been updated.
Status: Needs review = Approved
For more details, see:
This project is now hosted on https://github.com/OCA/account-invoice-reporting.
Please move your proposal there if you still want to merge it once fixed. This
guide may help you
https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub
Sorry Lorenzo didn't had the
Sorry for having a bit rushed the migration of this branch. One way or the
other we must replicate this MP as a PR on github, now. Can you do this and
putting this MP as reference, the PR will be merged fast as it was already
approved.
--
This MP introduced a bug
If first purchase order has no origin, it will raise
...
if not porder.origin in order_infos['origin'] and not order_infos['origin']
in porder.origin: TypeError: argument of type 'bool' is not iterable
because order_infos['origin'] will contains False
--
Public bug reported:
https://code.launchpad.net/~yann-papouin/ocb-addons/7.0-bug-1091268
-purchase-merge-no-origin-duplicates/+merge/210169
Introduced a bug:
When first purchase order has no origin, it will raise the following
error:
OpenERP Server Error Client Traceback (most recent call
Yannick Vaucher @ Camptocamp has proposed merging
lp:~camptocamp/ocb-addons/7.0-fix-1335887 into lp:ocb-addons.
Commit message:
Fix purchase order do_merge w/ 1st origin empty introduced on OCB on rev 10009
Requested reviews:
Yann Papouin (yann-papouin)
OpenERP Community Backports (ocb
Ok I though empty string were store as Null as zero value for integer. Just
check and it seems I'm wrong. I'll add a check instead of forcing to empty
string.
Thanks for the review
--
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1335887/+merge/225028
Your team OpenERP Community
@Ronald, I changed the fix according to your review
--
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1335887/+merge/225028
Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Ported on github
--
https://code.launchpad.net/~camptocamp/sale-wkfl/7.0-add-last_sale_price-yvr/+merge/223704
Your team Sale Core Editors is subscribed to branch lp:sale-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
The proposal to merge lp:~camptocamp/sale-wkfl/7.0-add-last_sale_price-yvr into
lp:sale-wkfl has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~camptocamp/sale-wkfl/7.0-add-last_sale_price-yvr/+merge/223704
--
Review: Approve code review, no tests
Just one inline comment
Otherwise LGTM
--
https://code.launchpad.net/~akretion-team/carriers-deliveries/7-split-default-option-state-from-deliv-method-dbl/+merge/224027
Your team Stock and Logistic Core Editors is subscribed to branch
Review: Approve
Ok failed my inline comment
l.102 field name=code readonly=1/
You can remove readonly here as you setted it on model
--
https://code.launchpad.net/~akretion-team/carriers-deliveries/7-split-default-option-state-from-deliv-method-dbl/+merge/224027
Your team Stock
** Also affects: ocb-addons
Importance: Undecided
Status: New
** Also affects: ocb-addons/7.0
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of OpenERP
Community Backports, which is subscribed to OpenERP Community Backports
Yannick Vaucher @ Camptocamp has proposed merging
lp:~camptocamp/ocb-addons/7.0-fix-1311087-rgo into lp:ocb-addons.
Requested reviews:
OpenERP Community Backports (ocb)
Related bugs:
Bug #1311087 in OpenERP Community Backports (Addons): mail: 'Send a message'
generates excessively slow
Yannick Vaucher @ Camptocamp has proposed merging
lp:~camptocamp/ocb-addons/7.0-fix-1223922-yvr into lp:ocb-addons.
Requested reviews:
OpenERP Community Backports (ocb)
Related bugs:
Bug #1223922 in OpenERP Community Backports (Addons): [7.0] account -
selection of analytic account
** Also affects: ocb-addons
Importance: Undecided
Status: New
** Changed in: ocb-addons
Status: New = Fix Committed
** Also affects: ocb-addons/7.0
Importance: Undecided
Status: Fix Committed
--
You received this bug notification because you are a member of OpenERP
Review: Approve code review no tests
LGTM thanks
--
https://code.launchpad.net/~camptocamp/stock-logistic-barcode/7.0-fix_1330459-sge+afe/+merge/223866
Your team Stock and Logistic Core Editors is subscribed to branch
lp:stock-logistic-barcode.
--
Mailing list:
Review: Approve code review, test
LGTM
Thanks
--
https://code.launchpad.net/~camptocamp/account-budgeting/vre-budget-responsible-group/+merge/223705
Your team Account Core Editors is subscribed to branch lp:account-budgeting.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
The proposal to merge lp:~agilebg/account-invoice-report/7_fix_1221570 into
lp:account-invoice-report has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~agilebg/account-invoice-report/7_fix_1221570/+merge/223781
--
Review: Approve code review, no test
Thanks
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:
Thanks again Lorenzo, will be a time saver.
--
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:
The proposal to merge lp:~agilebg/sale-reports/7_fix_1331541 into
lp:sale-reports has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~agilebg/sale-reports/7_fix_1331541/+merge/223871
--
Review: Approve code review
--
https://code.launchpad.net/~agilebg/sale-reports/7_fix_1331541/+merge/223871
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:sale-reports.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Thanks for the patch
--
https://code.launchpad.net/~agilebg/sale-reports/7_fix_1331541/+merge/223871
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:sale-reports.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Fixed in merge
--
https://code.launchpad.net/~agilebg/sale-reports/7_fix_1331541/+merge/223871
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:sale-reports.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
Review: Approve code review
As itself base_delivery_carrier_label module only provide an interface class,
this method here makes sense.
LGTM
--
https://code.launchpad.net/~akretion-team/carriers-deliveries/7-base-delivery-carrier-dbl/+merge/221226
Your team Stock and Logistic Core Editors is
This has been fixed by Pedro, some revision were overwritten last week and are
now back in OCB.
Thanks for noticing it Stéphane
--
https://code.launchpad.net/~camptocamp/account-financial-report/7.0-sync-afr_webkit-with-report_webkit/+merge/223058
Your team Account Report Core Editors is
The proposal to merge lp:~camptocamp/sale-wkfl/7.0-add-last_sale_price-yvr into
lp:sale-wkfl has been updated.
Description changed to:
Add a module to show last price defined for customer on sale order line
For more details, see:
Yannick Vaucher @ Camptocamp has proposed merging
lp:~camptocamp/sale-wkfl/7.0-add-last_sale_price-yvr into lp:sale-wkfl.
Commit message:
[ADD] module last_sale_price - This module provides new columns in sale order
line in order to let the sellerman know if a customer already ordrered
Thanks for the review.
I just fixed the __openerp__.py
--
https://code.launchpad.net/~camptocamp/sale-wkfl/7.0-add-last_sale_price-yvr/+merge/223704
Your team Sale Core Editors is subscribed to branch lp:sale-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to
Yannick Vaucher @ Camptocamp has proposed merging
lp:~camptocamp/ocb-addons/7.0-fix-github-650 into lp:ocb-addons.
Requested reviews:
OpenERP Community Backports (ocb)
For more details, see:
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-github-650/+merge/223727
Official MP
Thanks for the review Ana
My point about title_bank_account is that most of Models have standard `name`
and `description` fields.
I agree having a second field description is quiet useful to add note on the
object when you open it.
However as res.partner.bank is already a bank account, naming
Yannick Vaucher @ Camptocamp has proposed merging
lp:~camptocamp/account-financial-report/7.0-fix-1319109 into
lp:account-financial-report.
Requested reviews:
Account Report Core Editors (account-report-core-editor)
For more details, see:
https://code.launchpad.net/~camptocamp/account
fixed except: - except Exception:
--
https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-threaded-dispatch-label-generation/+merge/215184
Your team Stock and Logistic Core Editors is requested to review the proposed
merge of
Diff comments:
=== modified file 'delivery_carrier_label_dispatch/__openerp__.py'
--- delivery_carrier_label_dispatch/__openerp__.py2014-05-14 14:12:12
+
+++ delivery_carrier_label_dispatch/__openerp__.py2014-05-14 14:12:13
+
@@ -27,7 +27,7 @@
'depends':
Review: Needs Information
Doesn't the module firstname_display_name_trigger already does this job?
There is already a trigger in it to recompute display_name.
--
https://code.launchpad.net/~therp-nl/partner-contact-management/7.0_1184983_compute_display_name/+merge/220629
Your team Partner and
Review: Disapprove
firstname_display_name_trigger should already fix this issue.
Thus I disapprove it.
--
https://code.launchpad.net/~therp-nl/partner-contact-management/7.0_1184983_compute_display_name/+merge/220629
Your team Partner and Contact Core Editors is subscribed to branch
Review: Needs Fixing
Aggreed with Raphaël, Please provide a bug report and the same MP on official
branch
Thus the bug report will link ocb and official branch fixes.
Cheers
--
https://code.launchpad.net/~akretion-team/ocb-addons/70-addons-fix-payment-acls-bank/+merge/208021
Your team
Review: Approve code review, no test
LGTM
--
https://code.launchpad.net/~camptocamp/ocb-addons/ocb-7.0-fix_1302630_document_search_order_by-rde/+merge/214486
Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons.
--
Mailing list:
Review: Approve code review, no test
It should be fine. If employee has no department_id, employee.
Employee.department_id will return browse_null
and browse_null.id will return False
--
https://code.launchpad.net/~jb.eficent/department-mgmt/department-mgmt-bugfix-1296055/+merge/212286
Your
The proposal to merge
lp:~jb.eficent/department-mgmt/department-mgmt-bugfix-1296055 into
lp:department-mgmt has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~jb.eficent/department-mgmt/department-mgmt-bugfix-1296055/+merge/212286
--
Review: Approve
Thanks for the changes Pedro
LGTM
--
https://code.launchpad.net/~pedro.baeza/account-financial-tools/7.0-account-renumber-menu-fix/+merge/214832
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-financial-tools.
--
Mailing list:
The proposal to merge
lp:~camptocamp/account-financial-tools/7.0-fix-l10n_fr_siret-fix-lp1320918-rde
into lp:account-financial-tools has been updated.
Status: Needs review = Approved
For more details, see:
Review: Approve
--
https://code.launchpad.net/~therp-nl/ocb-web/7.0_lp1261332_better_report_name/+merge/219901
Your team OpenERP Community Backports is subscribed to branch lp:ocb-web.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to :
The proposal to merge
lp:~pedro.baeza/account-financial-tools/7.0-account_chart_update-enhanced into
lp:account-financial-tools has been updated.
Status: Needs review = Approved
For more details, see:
The proposal to merge
lp:~camptocamp/carriers-deliveries/7.0-associate-attachment-and-picking_rde
into lp:carriers-deliveries has been updated.
Status: Needs review = Approved
For more details, see:
Review: Needs Fixing
I set as need fixing after Frederic's comment
--
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
Review: Approve code review, no test
LGTM
thanks for the contrib!
--
https://code.launchpad.net/~acsone-openerp/account-financial-report/7.0-bug-1312732-lmi/+merge/217240
Your team Account Report Core Editors is subscribed to branch
lp:account-financial-report.
--
Mailing list:
Review: Needs Information
You are right forgot that sudden move from Odoo
At least it fixes this issue in 7.0 but this would be more effort when porting
to 8.0.
Thus I revert my review as we need to find an other way of ensuring values are
defined.
BTW shouldn't store be computed when
Review: Needs Fixing test + code review
Used it to create a config and works well.
Code review
l.161 + l.163 + l.172 please remove commented print line
l.226 no need to quote me for this code I took it from
addons/account/res_config.py
--
** Changed in: ocb-addons/7.0
Status: Fix Committed = Fix Released
--
You received this bug notification because you are a member of OpenERP
Community Backports, which is subscribed to OpenERP Community Backports
(Addons).
https://bugs.launchpad.net/bugs/1319109
Title:
webkit_report -
** Changed in: ocb-addons/7.0
Status: Fix Committed = Fix Released
--
You received this bug notification because you are a member of OpenERP
Community Backports, which is subscribed to OpenERP Community Backports
(Addons).
https://bugs.launchpad.net/bugs/1290820
Title:
report_webkit
Review: Approve code review and test
still you can remove l.223
Otherwise LGTM
--
https://code.launchpad.net/~akretion-team/server-env-tools/module-configuration-helper-dbl/+merge/220392
Your team Server Environment And Tools Core Editors is subscribed to branch
lp:server-env-tools.
--
Review: Approve code review, no test
LGTM
thanks!
--
https://code.launchpad.net/~agilebg/hr-timesheet/imp_hr_attendance_analysis_roundings/+merge/206493
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:hr-timesheet.
--
Mailing list:
Review: Approve code review, no test
Seems fine that way ondelete=cascade seems also a better choice.
Thanks!
LGTM
--
https://code.launchpad.net/~pedro.baeza/account-financial-tools/7.0-account_chart_update-enhanced/+merge/212074
Your team OpenERP Community Reviewer/Maintainer is subscribed to
Here is the reference for this community Guideline.
https://doc.openerp.com/contribute/05_developing_modules/#modules-description
This has the advantage to show who contributed directly on apps.odoo.com as we
don't have such field in manifest.
--
Here is an exemple:
http://bazaar.launchpad.net/~report-print-send-core-editors/report-print-send/7.0/view/head:/base_report_to_printer/__openerp__.py
--
https://code.launchpad.net/~numerigraphe-team/purchase-wkfl/7.0-add-purchase_delivery_split_date/+merge/211374
Your team Purchase Core Editors
Review: Approve code review, no test
Perfect, thanks for the changes Lionel
--
https://code.launchpad.net/~numerigraphe-team/purchase-wkfl/7.0-add-purchase_delivery_split_date/+merge/211374
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
--
Mailing list:
Review: Approve code review, no test
LGTM
Thanks
--
https://code.launchpad.net/~camptocamp/account-financial-tools/7.0-fix-l10n_fr_siret-fix-lp1320918-rde/+merge/220090
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:account-financial-tools.
--
Mailing list:
Review: Approve code review, no test
LGTM
--
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 :
The proposal to merge lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into
lp:purchase-wkfl has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745
--
Review: Approve code review, no test
LGTM
Cheers.
--
https://code.launchpad.net/~luc-demeyer/openerp-reporting-engines/7.0-report_xls-tz_fix/+merge/219267
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch
lp:openerp-reporting-engines.
--
Mailing list:
Review: Needs Fixing
Please don't add AUTHOR.txt file and add Contributors in __openerp__.py
For who did exaclty what there is the CVS
--
https://code.launchpad.net/~numerigraphe-team/purchase-wkfl/7.0-add-purchase_delivery_split_date/+merge/211374
Your team Purchase Core Editors is subscribed
Already merged
--
https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-fix-1319391/+merge/219530
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 :
1 - 100 of 668 matches
Mail list logo