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][IMP] sale_order_general_discount: apply discount product optin #1812

Conversation

petrus-v
Copy link

@petrus-v petrus-v commented Dec 8, 2021

My customer has hundred lines in sale order, he wants apply discount only on some kind of products according his business rules. As today he set general discoung and move back to zero each lines manually.

The purpose of this PR is to improve sale_order_general_discount module in order to add an option on product to define if the general discount should be applied on sale order lines according linked product.

@petrus-v
Copy link
Author

petrus-v commented Dec 8, 2021

Hi @OCA/crm-sales-marketing-maintainers cc @sbidoul not sure what should we do to makes runboat happy on this repo ? is there already some stuff managed about this use case ?

@sbidoul
Copy link
Member

sbidoul commented Dec 15, 2021

@petrus-v let's discuss handling of conflicting modules in runboat in sbidoul/runboat#36

Copy link
Contributor

@ferran-S73 ferran-S73 left a comment

Choose a reason for hiding this comment

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

Tested locally. Functional and code review

@simahawk
Copy link
Contributor

@sergio-teruel ping :)

Copy link
Contributor

@JordiMForgeFlow JordiMForgeFlow 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 LGTM

@petrus-v
Copy link
Author

petrus-v commented Nov 18, 2023

As far I understand, someone else develop something similar that has been merged in the mean time #2685 using a domain in settings. So, I'm closing this PR ! Feel free to adapt it and create a new PR if you feel needs user be able to set it on product directly

@petrus-v petrus-v closed this Nov 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants