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

[MIG] stock_by_warehouse_mrp_bom: Migration to 15.0 T#52780 #1520

Closed agarzaarvizu closed 2 years ago

agarzaarvizu commented 2 years ago

This PR refers to task#52780

agarzaarvizu commented 2 years ago

@CarmenMiranda @luisg123v please review.

agarzaarvizu commented 2 years ago

@luisg123v Done.

agarzaarvizu commented 2 years ago

@luisg123v @CarmenMiranda done.

CarmenMiranda commented 2 years ago

@agarzaarvizu In the image, it seems that there is no warehouse_id in the line, is that Ok?

Maybe if there is no warehouse there can be a change in the message so it doesn't say "The stock in: that is immediately..." @luisg123v What do you think?

agarzaarvizu commented 2 years ago

@CarmenMiranda @luisg123v Done. I had to added a view cause the product in the BoM line was editable, so I inherited the view to change that attribute. Now is working correctly.

CarmenMiranda commented 2 years ago

@CarmenMiranda @luisg123v Done. I had to added a view cause the product in the BoM line was editable, so I inherited the view to change that attribute. Now is working correctly.

Ohh ok, it was only editable with the tree view, is Ok for me but I don't know if that is Ok functionally @luisg123v what do you think?

Just update the translations, remember that we change the view, and in that change, the span was added so the term changed

agarzaarvizu commented 2 years ago

@CarmenMiranda @luisg123v Name of the view changed and new translation file added.