Re: [Openerp-community-reviewer] [Merge] lp:~the-clone-master/oemedical/oemedical_refactoring_step_I into lp:oemedical

2014-08-21 Thread Alexandre Fayolle - camptocamp
Review: Resubmit Hello, The management of the project has moved to Github: https://github.com/OCA/vertical-medical Please migrate your merge proposal to Github. You may want to check https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub for an explanation on

Re: [Openerp-community-reviewer] [Merge] lp:~the-clone-master/oemedical/oemedical_refactoring_step_I into lp:oemedical

2014-03-17 Thread Yannick Vaucher @ Camptocamp
Review: Needs Fixing code review l.391 could you search the group with it's xml id group_oemedical_doctor instead of the name ? Be careful, in xml you have a mix of spaces and tabs keep only spaces. --

Re: [Openerp-community-reviewer] [Merge] lp:~the-clone-master/oemedical/oemedical_refactoring_step_I into lp:oemedical

2014-01-20 Thread Pedro Manuel Baeza
Review: Needs Information Hi, Mario, Thanks for your great efforts to bring this project to a usable state. Waiting for a deeper review, I have seen something that flash me and want you to tell: you told about a new module called oemedical_first_name to split personal names in two, but this

Re: [Openerp-community-reviewer] [Merge] lp:~the-clone-master/oemedical/oemedical_refactoring_step_I into lp:oemedical

2014-01-20 Thread Mario Arias
Hi Pedro, In fact it is based on that one. It just adds inherited fields on new views... Regards El 20/01/2014 04:09, Pedro Manuel Baeza pedro.ba...@gmail.com escribió: Review: Needs Information Hi, Mario, Thanks for your great efforts to bring this project to a usable state. Waiting for a