Skip to content

[18.0] [MIG] sale_commission_pricelist: Migration to 18.0 + rename sale_commission_pricelist_oca#638

Open
Borruso wants to merge 30 commits intoOCA:18.0from
Borruso:18.0-mig-sale_commission_pricelist_oca
Open

[18.0] [MIG] sale_commission_pricelist: Migration to 18.0 + rename sale_commission_pricelist_oca#638
Borruso wants to merge 30 commits intoOCA:18.0from
Borruso:18.0-mig-sale_commission_pricelist_oca

Conversation

@Borruso
Copy link
Copy Markdown
Contributor

@Borruso Borruso commented Aug 29, 2025

No description provided.

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot migration sale_commission_pricelist

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 4, 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 Jan 4, 2026
@odooNextev
Copy link
Copy Markdown

@Borruso can you relaunch tests?
Do you remember what was the problem?
I guess we're using this without problem, but it's been a while and we don't remember it's yours or the previous one.

@Borruso Borruso force-pushed the 18.0-mig-sale_commission_pricelist_oca branch from 1790e6a to 612c332 Compare January 8, 2026 16:23
"license": "AGPL-3",
"depends": ["sale_commission_oca"],
"data": ["views/product_pricelist_view.xml"],
"pre_init_hook": "_sale_commission_pricelist_oca_pre_init_hook",

This comment was marked as resolved.

@Borruso Borruso force-pushed the 18.0-mig-sale_commission_pricelist_oca branch from 612c332 to a2d5779 Compare January 8, 2026 16:47
Copy link
Copy Markdown

@odooNextev odooNextev left a comment

Choose a reason for hiding this comment

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

account_commission changed into account_commission_oca in the meantime


from odoo.tests.common import Form

from odoo.addons.account_commission.tests.test_account_commission import (
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
from odoo.addons.account_commission.tests.test_account_commission import (
from odoo.addons.account_commission_oca.tests.test_account_commission import (

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@github-actions github-actions Bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 11, 2026
@Borruso Borruso force-pushed the 18.0-mig-sale_commission_pricelist_oca branch from a2d5779 to 78f62a3 Compare January 12, 2026 08:11
Copy link
Copy Markdown

@odooNextev odooNextev left a comment

Choose a reason for hiding this comment

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

LGTM

@JordiMForgeFlow
Copy link
Copy Markdown
Contributor

@Borruso you should split one commit for the migration and one for the renaming. Also, the renaming should apply to all the history. You can see instructions here: #610 (comment)

And you can also see how it was done for other commission modules: https://github.com/OCA/commission/pull/610/commits

carlosdauden and others added 15 commits March 24, 2026 13:13
Currently translated at 100.0% (3 of 3 strings)

Translation: commission-12.0/commission-12.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-12-0/commission-12-0-sale_commission_pricelist/it/
Currently translated at 100.0% (3 of 3 strings)

Translation: commission-12.0/commission-12.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-12-0/commission-12-0-sale_commission_pricelist/zh_CN/
Currently translated at 100.0% (3 of 3 strings)

Translation: commission-12.0/commission-12.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-12-0/commission-12-0-sale_commission_pricelist/pt/
Currently translated at 100.0% (3 of 3 strings)

Translation: commission-12.0/commission-12.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-12-0/commission-12-0-sale_commission_pricelist/pt_BR/
Currently translated at 100.0% (3 of 3 strings)

Translation: commission-12.0/commission-12.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-12-0/commission-12-0-sale_commission_pricelist/pt_BR/
ibuioli and others added 15 commits March 24, 2026 13:13
Currently translated at 100.0% (3 of 3 strings)

Translation: commission-13.0/commission-13.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-13-0/commission-13-0-sale_commission_pricelist/es_AR/
Currently translated at 100.0% (3 of 3 strings)

Translation: commission-14.0/commission-14.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-14-0/commission-14-0-sale_commission_pricelist/it/
Currently translated at 100.0% (3 of 3 strings)

Translation: commission-14.0/commission-14.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-14-0/commission-14-0-sale_commission_pricelist/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: commission-14.0/commission-14.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-14-0/commission-14-0-sale_commission_pricelist/
Currently translated at 100.0% (6 of 6 strings)

Translation: commission-14.0/commission-14.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-14-0/commission-14-0-sale_commission_pricelist/it/
Currently translated at 100.0% (6 of 6 strings)

Translation: commission-14.0/commission-14.0-sale_commission_pricelist
Translate-URL: https://translation.odoo-community.org/projects/commission-14-0/commission-14-0-sale_commission_pricelist/es/
@Borruso Borruso force-pushed the 18.0-mig-sale_commission_pricelist_oca branch from 1e80a30 to e456129 Compare March 24, 2026 12:13
@Borruso
Copy link
Copy Markdown
Contributor Author

Borruso commented Mar 24, 2026

@Borruso you should split one commit for the migration and one for the renaming. Also, the renaming should apply to all the history. You can see instructions here: #610 (comment)

And you can also see how it was done for other commission modules: https://github.com/OCA/commission/pull/610/commits

can you check?

@JordiMForgeFlow
Copy link
Copy Markdown
Contributor

@Borruso looks good! one question, why the init hook?

@OCA-git-bot
Copy link
Copy Markdown
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.