search

OCA / stock-logistics-barcode

https://odoo-community.org/psc-teams/logistics-18
GNU Affero General Public License v3.0
160 stars 330 forks source link

[14.0] [FIX] stock_barcodes: restore read_inventory to old version #582

Closed renda-dev closed 7 months ago

renda-dev commented 8 months ago

supersedes https://github.com/OCA/stock-logistics-barcode/pull/569

Fixes https://github.com/OCA/stock-logistics-barcode/issues/565 introduced by https://github.com/OCA/stock-logistics-barcode/pull/560

also, added tests that would fail without the latest commit.

francesco-ooops commented 8 months ago

@dalonsod good for you? :)

dalonsod commented 8 months ago

@alexis-via this solves #565 could you review?

francesco-ooops commented 8 months ago

@pedrobaeza @sergio-teruel this solves an issue introduced here: https://github.com/OCA/stock-logistics-barcode/pull/560#issuecomment-1867968119 , could you take a look? thanks!

pedrobaeza commented 8 months ago

If others confirmed it's OK, we can merge, but we don't work in this version with this.

OCA-git-bot commented 7 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

francesco-ooops commented 7 months ago

@pedrobaeza please proceed with merge, thanks!

pedrobaeza commented 7 months ago

/ocabot merge major

OCA-git-bot commented 7 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-582-by-pedrobaeza-bump-major, awaiting test results.

OCA-git-bot commented 7 months ago

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