search

OCA / sale-reporting

GNU Affero General Public License v3.0
47 stars 187 forks source link

[IMP] sale_order_product_recommendation_product_sold_by_delivery_week: display qty on hand #244

Closed yajo closed 8 months ago

yajo commented 8 months ago

If you care about last delivered items, you probably care about how many items you have on hand.

This improvement displays that info.

@moduon MT-4472

OCA-git-bot commented 8 months ago

Hi @chienandalu, some modules you are maintaining are being modified, check this out!

Shide commented 8 months ago

praise: Cool feature, this info is always welcome

suggestion: Is it possible to display the same widget? Gives more information, small width and same appearance.

If it's not possible, it's perfect for me

yajo commented 8 months ago

suggestion: Is it possible to display the same widget? Gives more information, small width and same appearance.

It seems tightly coupled with the sale.order.line model. I added this to the roadmap, and we can leave the current version as a MVP.

yajo commented 8 months ago

could we change label from Quantity yo Qty?

Changed.

yajo commented 8 months ago

@sergio-teruel @chienandalu any comments on this one?

yajo commented 8 months ago

I just pushed the roadmap change said in https://github.com/OCA/sale-reporting/pull/244#issuecomment-1857549346 that I forgot to commit.

/ocabot merge minor

OCA-git-bot commented 8 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-244-by-yajo-bump-minor, awaiting test results.

OCA-git-bot commented 8 months ago

Congratulations, your PR was merged at b4fa2743bc2474efe29e384d523e20fac30f9b7d. Thanks a lot for contributing to OCA. ❤️