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_by_warehouse: display stock by warehouse in widget T#79454 #1651

Closed xmglord closed 4 months ago

xmglord commented 5 months ago

There was an error when trying to load the stock by warehouse widget caused by the incorrectly use of onWillUpdateProps that was change in 1, this commit fixes that error and also adapt the stock by warehouse field to be computed. Finally, some styles were adapted to have a better display and to use recent version of bootstrap.

xmglord commented 5 months ago

@rolandojduartem @luisg123v please review.

rolandojduartem commented 5 months ago

Screenshot from 2024-05-03 10-28-30

Screenshot from 2024-05-03 10-28-22

Functionally it does not work :(

xmglord commented 5 months ago

Screenshot from 2024-05-03 10-28-30

Screenshot from 2024-05-03 10-28-22

Functionally it does not work :(

For me it is working: image :c

rolandojduartem commented 5 months ago

Functionally it is OK. @xmglord it did not worked because you had renamed the js file, but you did not changed the manifest, so it was loading nothing. It is Ok functionally

rolandojduartem commented 5 months ago

Just one comment more, the spanish translation was not updated during the migration process. @xmglord

cc @luisg123v

PD. After that, @luisg123v could merge this PR whenever he wants, I think it LGTM

xmglord commented 4 months ago

@luisg123v already update the translation file so this is ready to merge.

cc: @rolandojduartem