Skip to content

Conversation

@bizzappdev
Copy link

Dependency PR for module product_harmonized_system: #317

@bizzappdev bizzappdev force-pushed the 19.0-mig-product_harmonized_system_delivery-BAD branch from d074812 to 1168949 Compare December 1, 2025 11:16
@bizzappdev bizzappdev marked this pull request as ready for review December 1, 2025 11:23
@bizzappdev bizzappdev force-pushed the 19.0-mig-product_harmonized_system_delivery-BAD branch from 1168949 to 9f1d7bc Compare December 8, 2025 06:27
alexis-via and others added 25 commits December 8, 2025 12:04
This commit is the "followup" of the change to company_dependent=False
on the fields hs_code_id on product.template and product.category.
Add groupby on hs_code_id on product.template search view
hs.code is now company_id=False by default
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: intrastat-extrastat-15.0/intrastat-extrastat-15.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-15-0/intrastat-extrastat-15-0-product_harmonized_system_delivery/
Currently translated at 100.0% (5 of 5 strings)

Translation: intrastat-extrastat-16.0/intrastat-extrastat-16.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-16-0/intrastat-extrastat-16-0-product_harmonized_system_delivery/hr/
Currently translated at 100.0% (3 of 3 strings)

Translation: intrastat-extrastat-16.0/intrastat-extrastat-16.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-16-0/intrastat-extrastat-16-0-product_harmonized_system_delivery/es/
Currently translated at 100.0% (5 of 5 strings)

Translation: intrastat-extrastat-16.0/intrastat-extrastat-16.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-16-0/intrastat-extrastat-16-0-product_harmonized_system_delivery/fr/
Currently translated at 100.0% (5 of 5 strings)

Translation: intrastat-extrastat-16.0/intrastat-extrastat-16.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-16-0/intrastat-extrastat-16-0-product_harmonized_system_delivery/es/
Currently translated at 100.0% (5 of 5 strings)

Translation: intrastat-extrastat-16.0/intrastat-extrastat-16.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-16-0/intrastat-extrastat-16-0-product_harmonized_system_delivery/it/
[FIX] change no-stored field

[ADD] add contributor
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: intrastat-extrastat-18.0/intrastat-extrastat-18.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-18-0/intrastat-extrastat-18-0-product_harmonized_system_delivery/
Currently translated at 100.0% (5 of 5 strings)

Translation: intrastat-extrastat-18.0/intrastat-extrastat-18.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-18-0/intrastat-extrastat-18-0-product_harmonized_system_delivery/it/
Currently translated at 100.0% (5 of 5 strings)

Translation: intrastat-extrastat-18.0/intrastat-extrastat-18.0-product_harmonized_system_delivery
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-18-0/intrastat-extrastat-18-0-product_harmonized_system_delivery/it/
@bizzappdev bizzappdev force-pushed the 19.0-mig-product_harmonized_system_delivery-BAD branch from 9f1d7bc to 7f99fb1 Compare December 8, 2025 06:34
@bizzappdev bizzappdev marked this pull request as draft December 8, 2025 06:59
@bizzappdev bizzappdev force-pushed the 19.0-mig-product_harmonized_system_delivery-BAD branch 2 times, most recently from 5e0c699 to 408d071 Compare December 8, 2025 07:28
@bizzappdev bizzappdev marked this pull request as ready for review December 8, 2025 07:32
Copy link

@twalter-c2c twalter-c2c left a comment

Choose a reason for hiding this comment

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

Looks good, thank you for extending the test coverage. Only one small remark - please use the [IMP] tag instead of the [ADD] one for the commit containing additional tests (you can check the relevant guidelines here.

@bizzappdev bizzappdev force-pushed the 19.0-mig-product_harmonized_system_delivery-BAD branch from 408d071 to 6e27c95 Compare December 11, 2025 05:51
@bizzappdev
Copy link
Author

@twalter-c2c Changes in commit msg is done.

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

Comment on lines 13 to 17
<field name="arch" type="xml">
<field name="hs_code" position="attributes">

Choose a reason for hiding this comment

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

While checking in the UI, I noticed that there is an orphaned "Traceability" group in the product view. This has been made shown in v19. I suppose that we may want to hide it.

Suggested change
<field name="arch" type="xml">
<field name="hs_code" position="attributes">
<field name="arch" type="xml">
<xpath expr="//group[@name='traceability']" position="attributes">
<attribute name="invisible">1</attribute>
</xpath>
<field name="hs_code" position="attributes">

@OCA-git-bot
Copy link
Contributor

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). 🤖

@bizzappdev bizzappdev force-pushed the 19.0-mig-product_harmonized_system_delivery-BAD branch from 6e27c95 to b25053f Compare December 12, 2025 09:34
@twalter-c2c
Copy link

Thank you for applying the suggestion so quickly. It is all good now, in my opinion, well done!

@simahawk I guess we are good to merge this one as well. It would be great if you could do the magic 🙏🏻

@simahawk
Copy link

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-326-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2073500 into OCA:19.0 Dec 12, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e2dddbf. 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.