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

2014-06-26 Thread Alexandre Fayolle - camptocamp
Review: Needs Fixing code review, no tests please add automated tests. Additionally I'm interested in an explanation about the various comparison operators used : sometimes you use strict inequality, sometime -or-equal comparison (e.g. line 241 vs line 248). --

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

2014-06-04 Thread Benoit Guillot - http://www.akretion.com
Review: Approve code review no test LDTM -- https://code.launchpad.net/~akretion-team/account-closing/70-forecast-prepaid/+merge/210066 Your team Account Core Editors is requested to review the proposed merge of lp:~akretion-team/account-closing/70-forecast-prepaid into lp:account-closing. --

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

2014-03-08 Thread Alexis de Lattre
Alexis de Lattre has proposed merging lp:~akretion-team/account-closing/70-forecast-prepaid into lp:account-closing. Requested reviews: Account Core Editors (account-core-editors) For more details, see: