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

[16.0][IMP] product_packaging_multi_barcode: remove packaging_id of product.barcode on product form view #590

Closed QuocDuong1306 closed 2 months ago

QuocDuong1306 commented 7 months ago

Depend on:

Now users can not create a new barcode including packaging on General Information tab because it will raise this error (in PR above). So I remove the view to avoid confusion.

OCA-git-bot commented 6 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). 🤖

cyrilmanuel commented 6 months ago

hii @pedrobaeza , can you merge this PR please ?

pedrobaeza commented 6 months ago

It says it depends on https://github.com/OCA/stock-logistics-barcode/pull/563

cyrilmanuel commented 2 months ago

Hi @pedrobaeza the pr https://github.com/OCA/stock-logistics-barcode/pull/563 is merged, can you merge this one ?

thanks a lot

pedrobaeza commented 2 months ago

There are conflicts right now.

QuocDuong1306 commented 2 months ago

Hello @pedrobaeza , the PR is updated, could you have a look?

pedrobaeza commented 2 months ago

Well, I don't know the module, but I merge based on the reviews:

/ocabot merge minor

OCA-git-bot commented 2 months ago

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

OCA-git-bot commented 2 months ago

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