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

[16.0][ADD] delivery_package_sscc #902

Closed
wants to merge 2 commits into from

Conversation

thomaspaulb
Copy link

@thomaspaulb thomaspaulb commented Nov 11, 2024

This module allows to generate SSCC codes for packages automatically.

Eg stock.quant.package.name changes from PACK000001..000002 etc to an actual valid SSCC code, taken from a range that can be configured.

image

This also makes Odoo's stock.quant.package.valid_sscc flag True for any newly created package.

@thomaspaulb thomaspaulb marked this pull request as draft November 11, 2024 21:16
@thomaspaulb thomaspaulb force-pushed the 16.0-add-sequence_sscc branch 3 times, most recently from 693c003 to 3bc1a8f Compare November 11, 2024 22:32
This module allows to generate SSCC codes for packages automatically.
@thomaspaulb thomaspaulb force-pushed the 16.0-add-sequence_sscc branch from 3bc1a8f to e18b372 Compare November 11, 2024 22:50
@thomaspaulb thomaspaulb marked this pull request as ready for review November 11, 2024 22:55
@sebalix sebalix added this to the 16.0 milestone Dec 20, 2024
Copy link
Contributor

@sebalix sebalix left a comment

Choose a reason for hiding this comment

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

Nice addition, but should not this module lands in https://github.com/OCA/stock-logistics-tracking or https://github.com/OCA/stock-logistics-barcode ?
It's not linked directly to delivery carriers IMO, so the module should be renamed stock_quant_package_sscc for instance.

delivery_package_sscc/readme/USAGE.rst Outdated Show resolved Hide resolved
Co-authored-by: Sébastien Alix <sebalix@users.noreply.github.com>
@thomaspaulb
Copy link
Author

@sebalix You're probably right, I was confused as to where to place it myself. It seems to be linked to delivery in my mind because it's a tracking code for the packages that get sent out. Which of the two would you suggest to put it?

@sebalix
Copy link
Contributor

sebalix commented Dec 20, 2024

I would go for the tracking one, but let ask others opinions
cc @OCA/logistics-maintainers

Copy link
Contributor

@jbaudoux jbaudoux left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution. Looks good.
Move to tracking and rename like sebalix suggested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants