openerp-community-reviewer team mailing list archive

["Federico Manuel Echeverri Choux - \( Vauxoo \)" <echeverrifm@xxxxxxxxx>]

Hi, Pedro
Thanks for fhe Merge and i will do it ( one MP = one Merge )

Regards


----------------------------------
Federico Manuel Echeverri Choux
Vauxoo - OpenERP's Gold Partner
mobile: (+521) 444-1-400683
Tel: (+52) 444-8185539
skype: echeverrifm
web: http://www.vauxoo.com
twitter: @echeverrifm
           @vauxoo
On Nov 3, 2013 7:19 AM, "Pedro Manuel Baeza" <pedro.baeza@xxxxxxxxx> wrote:

> Review: Approve code review, no test
>
> Hi, Federico,
>
> Thank you very much for your contribution. I have checked it and seems OK,
> but for another time, it would be desirable not to mix different changes in
> one MP, because reviewing process is harder. For example, one MP with only
> the translation file can be almost approved blindly.
>
> Regards.
> --
> https://code.launchpad.net/~echeverrifm/oemedical/oemedical/+merge/192953
> You are the owner of lp:~echeverrifm/oemedical/oemedical.
>

-- 
https://code.launchpad.net/~echeverrifm/oemedical/oemedical/+merge/192953
Your team OEmedical Commiter is subscribed to branch lp:oemedical.