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_default_uom: Option to base the sale_price on sale_uom_id #2583

Closed
wants to merge 0 commits into from

Conversation

sistecem
Copy link

@sistecem sistecem commented Jun 23, 2023

[14.0][IMP] sale_default_uom: Option to base the sale_price on sale_uom_id
The default sale price was based on uom_id
An option to based on sale_uom_id ( when this is set ) is created with a boolean tick box. Making it easier and more accurate on decimals, useful on big quantities and rare uom Factors.

@OCA-git-bot
Copy link
Contributor

Hi @AshishHirapara,
some modules you are maintaining are being modified, check this out!

@sistecem sistecem force-pushed the 14.0 branch 4 times, most recently from 078aad7 to 947bbdf Compare June 28, 2023 13:55
@rousseldenis
Copy link
Sponsor Contributor

@sistecem Could you update your PR title and commit message to mention the changed module like [IMP] sale_default_uom: (...)? Thanks

@sistecem sistecem changed the title [IMP] Option to base the sale_price on sale_uom_id [IMP] sale_default_uom : Option to base the sale_price on sale_uom_id Jul 31, 2023
@rafaelbn rafaelbn changed the title [IMP] sale_default_uom : Option to base the sale_price on sale_uom_id [14.0][IMP] sale_default_uom: Option to base the sale_price on sale_uom_id Aug 7, 2023
@rafaelbn rafaelbn added this to the 14.0 milestone Aug 7, 2023
@rafaelbn
Copy link
Member

rafaelbn commented Aug 7, 2023

Hello @sistecem are you going to continue with this PR? Let me know. If no or nor answer we should close it. Thank you!

@sistecem
Copy link
Author

Thanks @rafaelbn , is that way ok ?
Or do I have to edit anything else ?

@rousseldenis
Copy link
Sponsor Contributor

@sistecem LGTM. Could you add a test for this change ?

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.

4 participants