Skip to content

Comments

[19.0][MIG] product_tags_code: Migration to 19.0#2091

Open
bizzappdev wants to merge 4 commits intoOCA:19.0from
BizzAppDev-Systems:19.0-mig-product_tags_code-BAD
Open

[19.0][MIG] product_tags_code: Migration to 19.0#2091
bizzappdev wants to merge 4 commits intoOCA:19.0from
BizzAppDev-Systems:19.0-mig-product_tags_code-BAD

Conversation

@bizzappdev
Copy link
Contributor

No description provided.

@bizzappdev bizzappdev force-pushed the 19.0-mig-product_tags_code-BAD branch from d5c38ab to dc22a12 Compare October 1, 2025 10:28
@bizzappdev bizzappdev marked this pull request as ready for review October 1, 2025 11:02
@rousseldenis
Copy link
Contributor

/ocabot migration product_tags_code

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Oct 6, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 6, 2025
72 tasks
product_tag = self.env["product.tag"].create(
{
"name": "Test Tag",
"product_template_ids": [(6, 0, [self.product_tmpl.id])],
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"product_template_ids": [(6, 0, [self.product_tmpl.id])],
"product_template_ids": Command.set([self.product_tmpl.id]),

Minor suggestion: please switch to Command.set([self.product_tmpl.id]) rather than using (6, 0, ids).

@@ -0,0 +1 @@
Allows to reference tag records trough a unique code.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Allows to reference tag records trough a unique code.
Allows to reference tag records through a unique code.

@bizzappdev bizzappdev force-pushed the 19.0-mig-product_tags_code-BAD branch from dc22a12 to 62284bb Compare January 8, 2026 06:42
@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). 🤖

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.

5 participants