Skip to content

Comments

[17.0][MIG] product_abc_classification: Migration to 17.0#1686

Open
fgaviiria wants to merge 24 commits intoOCA:17.0from
fgaviiria:17.0-mig-product_abc_classification
Open

[17.0][MIG] product_abc_classification: Migration to 17.0#1686
fgaviiria wants to merge 24 commits intoOCA:17.0from
fgaviiria:17.0-mig-product_abc_classification

Conversation

@fgaviiria
Copy link
Contributor

No description provided.

@fgaviiria fgaviiria force-pushed the 17.0-mig-product_abc_classification branch 5 times, most recently from 0bf3ec5 to d8e3c8b Compare July 18, 2024 00:43
@fgaviiria fgaviiria mentioned this pull request Jul 18, 2024
69 tasks
@rousseldenis
Copy link
Contributor

/ocabot migration product_abc_classification

Copy link
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

Code review

)
)

@api.onchange("product_tmpl_id")
Copy link

Choose a reason for hiding this comment

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

It would be nice to convert this onchange to a computed field on the product_id.

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 23, 2025
rvalyi
rvalyi previously approved these changes Mar 23, 2025
@rvalyi
Copy link
Member

rvalyi commented Mar 23, 2025

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-1686-by-rvalyi-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Mar 23, 2025
Signed-off-by rvalyi
@OCA-git-bot
Copy link
Contributor

@rvalyi The merge process could not be finalized, because command oca-gen-addon-readme --if-source-changed --org-name OCA --repo-name product-attribute --branch 17.0 --addons-dir /tmp/tmpknocilbd --commit failed with output:

Both .md and .rst found for DESCRIPTION. Please remove one of /tmp/tmpknocilbd/product_abc_classification/readme/DESCRIPTION.rst or /tmp/tmpknocilbd/product_abc_classification/readme/DESCRIPTION.md.

@rvalyi rvalyi dismissed their stale review March 23, 2025 21:22

Merge is failing

Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

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

@fgaviiria please remove the old .rst documentation files (see error message above). You can do this with by amending the penultimate commit with:
git rebase HEAD~3 -i
Choose to amend (e) the penultimate commit.
And then commit the old .rst files with the --amend option
Finish the rebase with gir rebase --continue
Push with the --force option.

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 30, 2025
@anothingguy
Copy link
Contributor

noticed this migration removed feature: setting profile on product category

@rousseldenis
Copy link
Contributor

/ocabot rebase

MiquelRForgeFlow and others added 24 commits September 2, 2025 15:11
Currently translated at 8.0% (6 of 75 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_abc_classification
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_abc_classification/es/
Currently translated at 100.0% (75 of 75 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_abc_classification
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_abc_classification/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: product-attribute-16.0/product-attribute-16.0-product_abc_classification
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_abc_classification/
Currently translated at 100.0% (74 of 74 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_abc_classification
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_abc_classification/it/
The test on unicity constraint is passing but the error log from the ORM
must be muted to not get a false negative.
@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 17.0.

@OCA-git-bot OCA-git-bot force-pushed the 17.0-mig-product_abc_classification branch from d8e3c8b to e38fad2 Compare September 2, 2025 15:11
Copy link
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

Ooops, @fgaviiria could you remove *.rst files ?

@github-actions
Copy link

github-actions bot commented Feb 1, 2026

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

migration needs review stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.