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.

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

[11.0][add] stock_barcodes_move_location #227

Merged
merged 2 commits into from Dec 5, 2019

Conversation

JordiBForgeFlow
Copy link
Sponsor Member

@JordiBForgeFlow JordiBForgeFlow commented Nov 13, 2019

This module introduces the ability to use barcode scanners within the stock_move_location wizard.

image

image

The PR depends on:

@JordiBForgeFlow
Copy link
Sponsor Member Author

@Tecnativa @sergio-teruel @lreficent @ageficent @NuriaMartinXifre @Joansis

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

Just tested it on runbot. Very cool complement for the stock_barcodes_* family! Thanks!

Copy link

@JoanSForgeFlow JoanSForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional review done, LGTM!

@LoisRForgeFlow LoisRForgeFlow force-pushed the 11.0-stock_barcodes_move_location branch from 1e0d43a to 2fff6ef Compare December 5, 2019 14:20
@LoisRForgeFlow
Copy link

I have just rebased and squashed a little bit the commits, this should be ready to merge.

Copy link

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM

@LoisRForgeFlow
Copy link

/ocabot merge

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 11.0-ocabot-merge-pr-227-by-lreficent-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 5, 2019
Signed-off-by lreficent
@OCA-git-bot OCA-git-bot merged commit 2fff6ef into OCA:11.0 Dec 5, 2019
@OCA-git-bot
Copy link
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 11.0-stock_barcodes_move_location branch December 5, 2019 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants