Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves into lp:ocb-addons/6.1

2014-03-10 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no test forward port of bug fix in 6.1 Thanks -- https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves/+merge/197753 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons/6.1. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves into lp:ocb-addons/6.1

2014-03-10 Thread Sandy Carter (http://www.savoirfairelinux.com)
Review: Needs Fixing code review, pep8 PEP8: l.11 W603 '' is deprecated, use '!=' l.14 E231 missing whitespace after ',' l.14 E231 missing whitespace after ':' -- https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves/+merge/197753 Your team OpenERP Community

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves into lp:ocb-addons/6.1

2014-03-10 Thread Yannick Vaucher @ Camptocamp
Missed the target with my comment :/ Already merged -- https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves/+merge/197753 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons/6.1. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves into lp:ocb-addons/6.1

2014-01-20 Thread noreply
The proposal to merge lp:~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves into lp:ocb-addons/6.1 has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves/+merge/197753 --

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves into lp:ocb-addons/6.1

2014-01-06 Thread Holger Brunn (Therp)
Review: Approve code review lines 8ff differ from the fix proposed against, but that's a good thing in my opinion -- https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves/+merge/197753 Your team OpenERP Community Backports Team is subscribed to branch

[Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves into lp:ocb-addons/6.1

2013-12-04 Thread Yann Papouin
Yann Papouin has proposed merging lp:~yann-papouin/ocb-addons/6.1-bug-1104766-ignore-canceled-moves into lp:ocb-addons/6.1. Requested reviews: OpenERP Community Backports Team (ocb) Related bugs: Bug #1104766 in OpenERP Community Backports (Addons): Product Return Canceled