search

Vauxoo / addons-vauxoo

All our modules related to developments that solves generic issues on Odoo, or that solve internal problems on Odoo Core, if something is here, maybe it is solving an issue in your company, try it and report what you see.
http://www.vauxoo.com
193 stars 288 forks source link

[FIX] stock_cost_segmentation: Get correct candidates based location … issue#7678 #1375

Closed keylor2906 closed 4 years ago

keylor2906 commented 4 years ago

…instead of warehouse.

[REF] stock_cost_segmentation: Added server action to fix existing moves logistic quantity to match the quants.

keylor2906 commented 4 years ago

@josemoralesp Could you review this, please?

cc @dsabrinarg

dsabrinarg commented 4 years ago

@keylor2906 aquí vamosa necesitar el Vo. Bo. de @hbto

keylor2906 commented 4 years ago

Hola @hbto

No hay video, pero lo voy a crear para añadirlo.

La acción no cambia quants, los quants se utilizan para modificar las cantidades logisticas en los moves. El campo de cantidades logisticas es un campo meramente informativo, no tiene relación alguna con contabilidad. Este cambio en los quants se debe a que actualmente el feature no esta funcionando de forma correcta, y en este PR se hace el fix al feature para que se haga la reducción de cantidades logisticas de forma correcta de los IN move cuando hay un OUT move donde correspondan sus ubicaciones (destino para IN y origen para OUT).

La acción de servidor es para corregir la data existente en base a dicho fix.

keylor2906 commented 4 years ago

@hbto

Me faltó añadir que el campo cantidades logisticas que es meramente informativo es custom y añadido en el modulo donde se estan realizando el cambio. Solo que este debe reflejar uniformidad con la información que representan los quants.

keylor2906 commented 4 years ago

Video del fix: https://youtu.be/ihKcmJwgaz0

keylor2906 commented 4 years ago

@dsabrinarg This is the video of the fix https://youtu.be/ihKcmJwgaz0 the PR is approved by Hbto

hbto commented 4 years ago

I have watched the video 👍

https://github.com/Vauxoo/addons-vauxoo/pull/1375#issuecomment-547658644

Please merge.

Regards.