openerp-community-reviewer team mailing list archive

[Nhomar - Vauxoo <nhomar@xxxxxxxxx>]

2013/10/31 Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>

> Review: Needs Information code review, no test
>
> Yeah, that's correct, Mario! You will use the second one as the most
> updated branch until these changes are merged.
>
> About the new submission, thank you for changing import format, but you
> still can replace osv by orm, that it's the new form.
>

+1 to this but can you share an example dude.

>
> Is it very difficult to fix the views you have commented?
>
I think is fine the way Mario did, comment clearly why because it is not
the objective of the branch, it can be done in a different approach, as we
are improving a lot of thing, temporary it is fine.

>
> Regards.
> --
>
> https://code.launchpad.net/~the-clone-master/oemedical/oemedical/+merge/193431
> You are requested to review the proposed merge of
> lp:~the-clone-master/oemedical/oemedical into lp:oemedical.
>



-- 
--------------------
Saludos Cordiales

Nhomar G. Hernandez M.
+58-414-4110269
Skype: nhomar00
Web-Blog: http://geronimo.com.ve
Servicios IT: http://vauxoo.com
Linux-Counter: 467724
Correos:
nhomar@xxxxxxxxxxxxxx
nhomar@xxxxxxxxxx
twitter @nhomar

https://code.launchpad.net/~the-clone-master/oemedical/oemedical/+merge/193431
Your team OEmedical Commiter is subscribed to branch lp:oemedical.