search

OCA / stock-logistics-barcode

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

[15.0][IMP] stock_barcodes_gs1: Refactoring. Use Odoo core module to decode GS1 barcodes #543

Closed sergio-teruel closed 9 months ago

sergio-teruel commented 10 months ago

Better process of AI's

cc @Tecnativa TT45427

ping @carlosdauden @CarlosRoca13 @dalonsod @etobella

Depends on:

carlosdauden commented 10 months ago

/ocabot rebase

OCA-git-bot commented 10 months ago

Sorry @carlosdauden you are not allowed to rebase.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

etobella commented 10 months ago

/ocabot rebase

OCA-git-bot commented 10 months ago

@etobella The rebase process failed, because command git push --force Tecnativa tmp-pr-543:15.0-IMP-stock_barcodes_gs1-refactoring failed with output:

remote: Permission to Tecnativa/stock-logistics-barcode.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/Tecnativa/stock-logistics-barcode/': The requested URL returned error: 403
pedrobaeza commented 10 months ago

On organization branches, only manual rebase and push can be done. This is due to GitHub forbidding it.

sergio-teruel commented 10 months ago

Rebase done... I have to fix tests without stock_barcodes_gs1_expiry module installed

OCA-git-bot commented 10 months ago

On my way to merge this fine PR! Prepared branch 15.0-ocabot-merge-pr-543-by-pedrobaeza-bump-major, awaiting test results.

OCA-git-bot commented 10 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-543-by-pedrobaeza-bump-major.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

pedrobaeza commented 10 months ago

/ocabot merge major

OCA-git-bot commented 10 months ago

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

OCA-git-bot commented 10 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-543-by-pedrobaeza-bump-major.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

pedrobaeza commented 10 months ago

/ocabot merge major

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

pedrobaeza commented 9 months ago

/ocabot merge major

OCA-git-bot commented 9 months ago

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

OCA-git-bot commented 9 months ago

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