Skip to content

Conversation

@rousseldenis
Copy link

In some flows like GS1, a barcode representation can contain multiple content (e.g.: product, lot, expiration date, ...).

In current flows, we allow only one record return from the searches based on barcode scan.

In order to change the behavior when some extra data is found in barcodes, we add the parsed result in the search result.

e.g.: For reception flows, if we scan a GS1 and the lot is not found, we want to pre-fill expiration date for instance

@OCA-git-bot
Copy link
Contributor

Hi @guewen, @sebalix, @simahawk,
some modules you are maintaining are being modified, check this out!

@rousseldenis rousseldenis force-pushed the 16.0-imp-shopfloor-parse-barcode-dro branch 2 times, most recently from 5889c0b to bd7b26b Compare June 27, 2025 09:31
@rousseldenis
Copy link
Author

@sebalix @jbaudoux

@rousseldenis
Copy link
Author

@jbaudoux

@rousseldenis
Copy link
Author

@lmignon @simahawk

return None

def expiration_date_from_scan(self, barcode):
# TODO
Copy link
Contributor

Choose a reason for hiding this comment

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

still TODO ?

Copy link
Author

Choose a reason for hiding this comment

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

That's following @simahawk

For the time being, we don't search for expiration dates but we need to have a handler to get a parse result for dates from GS1

@jbaudoux
Copy link
Contributor

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1001-by-jbaudoux-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d494061 into OCA:16.0 Jul 18, 2025
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@lmignon lmignon deleted the 16.0-imp-shopfloor-parse-barcode-dro branch July 18, 2025 14:08
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.

4 participants