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

14.0 mig sale_commission_formula #301

Merged
merged 31 commits into from
Oct 31, 2021

Conversation

fredzamoabg
Copy link
Contributor

@fredzamoabg fredzamoabg commented May 24, 2021

sale_commission_formula module migration from 13.0
(depends on sale_commission migration #302)

@fredzamoabg fredzamoabg mentioned this pull request May 24, 2021
5 tasks
@fredzamoabg fredzamoabg force-pushed the 14.0-mig-sale_commission_formula branch from caeae94 to 93dfbbd Compare May 26, 2021 06:57
@rousseldenis rousseldenis added this to the 14.0 milestone May 26, 2021
@SerM11
Copy link

SerM11 commented Jun 4, 2021

test funzionale ok

Nicola Malcontenti and others added 24 commits June 4, 2021 13:23
- 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
ibuioli and others added 7 commits June 4, 2021 13:23
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/
…es not translatable

- Put t-translation="off" for elements that shouldn't be translated
- Define name attributes to xpath elements for better inheritance
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/
Currently translated at 100.0% (14 of 14 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/
The invoice post() was done twice, which is unnecessary and with the
recent upstream changes raises an exception
@fredzamoabg fredzamoabg force-pushed the 14.0-mig-sale_commission_formula branch from 93dfbbd to 9f4c0a8 Compare June 4, 2021 11:24
@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 Oct 31, 2021
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 807b3db into OCA:14.0 Oct 31, 2021
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 815ef2e. 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
Labels
merged 🎉 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.