Skip to content
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

[15.0] sale_commission_formula: Migration to 15.0 #377

Merged
merged 38 commits into from
Dec 28, 2022

Conversation

AungKoKoLin1997
Copy link
Contributor

@AungKoKoLin1997 AungKoKoLin1997 commented Oct 25, 2022

No description provided.

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 15.0-mig-sale_commission_formula branch from 6566dbe to e026a5a Compare October 25, 2022 07:07
@HaraldPanten
Copy link

Hi @AungKoKoLin1997 will you continue with the migration?

THX

@AungKoKoLin1997
Copy link
Contributor Author

Hi @AungKoKoLin1997 will you continue with the migration?

Yes. The migration of commission modules are done. So, I am planning to work on these days.

@AungKoKoLin1997 AungKoKoLin1997 marked this pull request as draft December 23, 2022 09:28
@HaraldPanten
Copy link

Hi @AungKoKoLin1997 will you continue with the migration?

Yes. The migration of commission modules are done. So, I am planning to work on these days.

THX!

Nicola Malcontenti and others added 25 commits December 23, 2022 16:40
- Remove onchange hackings as they are not needed with new JS views.
- Extract common code between sales orders and invoices, creating 2 mixins.
- Adapt tests.
- Improve README with detailed explanations.
- README by fragments.
- Changed `settled` condition to a more suitable one, moving the condition to _skip_settlement.
- Improve interface for not removing editable option in lines: you have a new icon that popups
  a window with agents. This icon is available according certain conditions. Check README for
  more details.
- Add constraint for avoiding the modification of a settled commission.
- Add agent values when no other value is provided. This way, we add compatibility with
  modules like `sale_product_set` or `sale_order_variant_mgmt`.
* Add description to all models
* Adapt commission amount computation courtesy of Simone Rubi
* Adapt views
* Fixed dependencies in constraint
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: commission-12.0/commission-12.0-sale_commission_formula
Translate-URL: https://translation.odoo-community.org/projects/commission-12-0/commission-12-0-sale_commission_formula/

[UPD] Update sale_commission_formula.pot

[UPD] README.rst
Currently translated at 95.2% (20 of 21 strings)

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

Translation: commission-12.0/commission-12.0-sale_commission_formula
Translate-URL: https://translation.odoo-community.org/projects/commission-12-0/commission-12-0-sale_commission_formula/zh_CN/
Currently translated at 61.9% (13 of 21 strings)

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

Translation: commission-12.0/commission-12.0-sale_commission_formula
Translate-URL: https://translation.odoo-community.org/projects/commission-12-0/commission-12-0-sale_commission_formula/pt_BR/
Currently translated at 85.7% (18 of 21 strings)

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

Translation: commission-13.0/commission-13.0-sale_commission_formula
Translate-URL: https://translation.odoo-community.org/projects/commission-13-0/commission-13-0-sale_commission_formula/

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: commission-13.0/commission-13.0-sale_commission_formula
Translate-URL: https://translation.odoo-community.org/projects/commission-13-0/commission-13-0-sale_commission_formula/

[UPD] Update sale_commission_formula.pot
Currently translated at 100.0% (19 of 19 strings)

Translation: commission-13.0/commission-13.0-sale_commission_formula
Translate-URL: https://translation.odoo-community.org/projects/commission-13-0/commission-13-0-sale_commission_formula/es_AR/
OCA-git-bot and others added 5 commits December 23, 2022 16:40
Currently translated at 85.7% (12 of 14 strings)

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

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

Translation: commission-14.0/commission-14.0-sale_commission_formula
Translate-URL: https://translation.odoo-community.org/projects/commission-14-0/commission-14-0-sale_commission_formula/pt_BR/
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 15.0-mig-sale_commission_formula branch 2 times, most recently from e481014 to 5476d90 Compare December 23, 2022 10:12
@AungKoKoLin1997 AungKoKoLin1997 marked this pull request as ready for review December 23, 2022 10:21
@AungKoKoLin1997
Copy link
Contributor Author

@HaraldPanten Can you please review this PR?

@HaraldPanten
Copy link

@HaraldPanten Can you please review this PR?

Sure!

@luis-ron

Copy link
Contributor

@luis-ron luis-ron left a comment

Choose a reason for hiding this comment

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

LGTM

@pedrobaeza
Copy link
Member

/ocabot migration sale_commission_formula

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Dec 28, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 28, 2022
5 tasks
@OCA-git-bot
Copy link
Contributor

The migration issue (#328) has been updated to reference the current pull request.
however, a previous pull request was referenced : #359.
Perhaps you should check that there is no duplicate work.
CC : @olgamarcocb

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 15.0-mig-sale_commission_formula branch from bcfc1d0 to b9242ee Compare December 28, 2022 12:18
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.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-377-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c2a4aab into OCA:15.0 Dec 28, 2022
@OCA-git-bot
Copy link
Contributor

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