Samuel Macias - OSI

Results 17 comments of Samuel Macias - OSI

@max3903 @dreispt @AmmarOfficewalaSerpentCS

This PR requires #1150 to be merged first so the test can pass after a rebase

@max3903 @pedrobaeza I have a question. We would like to modify this module in order to use the existing field employee_number instead of the new field identification_id. To make this...

> I think that should be another module, not this one. The functionality would be the exact same, just making use of an existing field instead of adding a new...

@max3903 I have tested this. It works as expected

@AmmarOfficewalaSerpentCS Tests are still failing. Please fix these ASAP cc: @max3903

> Hello @SMaciasOSI and thanks for this work If i understand well the code, your module permits to quickly apply sale price to a product thanks to the sum of...

> Yep ok ! Two modules for two very differents usages ! (don't hesitate to review mine ;-)) It's seems to be quite a draft as you are fixing stuffs...