Skip to content

[18.0][MIG] commission + rename to commission_oca #610

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

Merged
merged 87 commits into from
Apr 28, 2025

Conversation

JordiMForgeFlow
Copy link
Contributor

@JordiMForgeFlow JordiMForgeFlow commented Apr 22, 2025

Supersedes #602 due to inactivity

sale_commission will need to be renamed to sale_commission_oca (#605), so we could rename the base module to commission_oca for alignment.

CC @alan196

AungKoKoLin1997 and others added 30 commits April 22, 2025 09:57
- Make specific top-level menu and permissions for commissions.
- Don't depend on account.
- Reorganize elements.
- Change some parts of the code to fit better ORM tools.
- Adjust permissions in children modules.
- Add <sheet> to some form views to get modern view.
This way, we can restrict which commissions we can select or be
automatically populate in the different documents.
Currently translated at 53.7% (57 of 106 strings)

Translation: commission-15.0/commission-15.0-commission
Translate-URL: https://translation.odoo-community.org/projects/commission-15-0/commission-15-0-commission/ja/
Currently translated at 61.3% (65 of 106 strings)

Translation: commission-15.0/commission-15.0-commission
Translate-URL: https://translation.odoo-community.org/projects/commission-15-0/commission-15-0-commission/ja/
Currently translated at 100.0% (106 of 106 strings)

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

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

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

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

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

Translation: commission-16.0/commission-16.0-commission
Translate-URL: https://translation.odoo-community.org/projects/commission-16-0/commission-16-0-commission/
Copy link

@JasminSForgeFlow JasminSForgeFlow 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
Contributor Author

@sebalix could we get your input in this one?

Copy link

@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.

Don't know well this module, but from a technical POV it loogs good. I agree on the renaming too.

For info Odoo EE is proposing the following module technical names:

  • partner_commission
  • sale_commission
  • sale_commission_margin
  • sale_commission_subscription

@pedrobaeza
Copy link
Member

/ocabot migration commission

Thanks for the migration and the renaming. I'm afraid the renaming you have done doesn't include the whole commit history, so on the next version, only the last commit will be rescued with the procedure.

For avoiding this, you have to rename the whole history, please execute the following commands in your branch:

git filter-branch --tree-filter 'if [ -d commission ]; then mv commission commission_oca; fi' HEAD
git rebase origin/18.0
git push -f

and it will be done for the whole commit history.

Another thing: for the PO/POT, you have to also rename the module on each translation string comment.

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Apr 28, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 28, 2025
6 tasks
@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). 🤖

@JordiMForgeFlow
Copy link
Contributor Author

@pedrobaeza it should now be ready :) thank you!

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Thanks!

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 4587c5c into OCA:18.0 Apr 28, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 511450f. Thanks a lot for contributing to OCA. ❤️

@MiquelRForgeFlow
Copy link

Shouldn't be done anything in OpenUpgrade?

@pedrobaeza
Copy link
Member

Yes, the renaming should happen also in OpenUpgrade, so you can proceed with the PRs. Thanks for being aware.

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.