Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[16.0][MIG] pos_product_multi_barcode: Migration to 16.0 #1037

Merged
merged 13 commits into from
Nov 16, 2023
Merged

[16.0][MIG] pos_product_multi_barcode: Migration to 16.0 #1037

merged 13 commits into from
Nov 16, 2023

Conversation

nguyenminhchien
Copy link
Contributor

No description provided.

@nguyenminhchien nguyenminhchien marked this pull request as draft August 11, 2023 04:19
@nilshamerlinck nilshamerlinck mentioned this pull request Aug 22, 2023
Closed
@nguyenminhchien nguyenminhchien marked this pull request as ready for review August 28, 2023 02:29
@luke-stdev001
Copy link

@nguyenminhchien ,

Happy to do a functional review on this as a bystander, do you have usage steps for this in the README.md?

@nguyenminhchien
Copy link
Contributor Author

@luke-stdev001
Hello, the module product_multi_barcode allows you to add many barcodes for a product (have a look at the attached image below).
the POS feature allows you to select a product (from the pos screen) via scanning its barcode.
now, this module allows you to scan any of barcodes you define in product form view
image

@legalsylvain
Copy link
Contributor

/ocabot migration pos_product_multi_barcode

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Sep 26, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Sep 26, 2023
Open
37 tasks
simahawk
simahawk approved these changes Oct 9, 2023
View reviewed changes
Copy link

@simahawk simahawk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG, just code review

pos_product_multi_barcode/readme/CONTRIBUTORS.rst Outdated
@@ -0,0 +1,4 @@
* Ooops404 <https://ooops404.com>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

funny that the original author @PierrickBrun is not here... can you fix this pls?

Pierrick Brun and others added 13 commits October 9, 2023 16:44
@nguyenminhchien
[ADD] pos_product_multi_barcode
a482e5b
@nguyenminhchien
[Misc] Set myself as maintainer
91069c5
@nguyenminhchien
[UPD] Update pos_product_multi_barcode.pot
d39c402
@OCA-git-bot @nguyenminhchien
[UPD] README.rst
02d2551
@OCA-git-bot @nguyenminhchien
[ADD] icon.png
053d571
@OCA-git-bot @nguyenminhchien
pos_product_multi_barcode 14.0.1.0.1
7644c19
@ilyasProgrammer @nguyenminhchien
[14.0][FIX] pos_product_multi_barcode: enable feature to search for p…
32c8914 
…roducts by multiple barcodes in search bar of POS.
@OCA-git-bot @nguyenminhchien
[UPD] README.rst
f61f0bf
@OCA-git-bot @nguyenminhchien
pos_product_multi_barcode 14.0.1.0.2
60ac201
@mymage @nguyenminhchien
Added translation using Weblate (Italian)
e0ee6cd
@mymage @nguyenminhchien
Translated using Weblate (Italian)
c8e7abd 
Currently translated at 100.0% (5 of 5 strings)

Translation: pos-14.0/pos-14.0-pos_product_multi_barcode
Translate-URL: https://translation.odoo-community.org/projects/pos-14-0/pos-14-0-pos_product_multi_barcode/it/
@nguyenminhchien
[IMP] pos_product_multi_barcode: black, isort, prettier
54e7e2e
@nguyenminhchien
[16.0][MIG] pos_product_multi_barcode: Migration to 16.0
9f79b04
@simahawk
Copy link

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1037-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a7315b8 into OCA:16.0 Nov 16, 2023
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simahawk simahawk simahawk approved these changes

@cyrilmanuel cyrilmanuel cyrilmanuel approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.
8 participants
@nguyenminhchien @luke-stdev001 @legalsylvain @simahawk @OCA-git-bot @cyrilmanuel @ilyasProgrammer @mymage