-
-
Notifications
You must be signed in to change notification settings - Fork 636
[16.0][MIG] pos_pricelist_technical #1130
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
Conversation
/ocabot migration pos_pricelist_technical |
d6a57aa
to
7f4f486
Compare
7f4f486
to
cd913cd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey !
Why not hiding technical pricelist directly on pos.config ?
thanks for this PR and migration
Do you mean, removing the pricelist from the field Considering
If you don't load the pricelist 2 in the pos.config, it will not be available in the PoS. As a result the computation of the pricelist 3 will fail, because information are not available. |
Ok i didn't know ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
functionnal and code review, thank you for this migration ✔️
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. |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1130-by-legalsylvain-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
Avoid to display in the point of sale pricelists that are used only to compute other pricelist
Co-authored-by: Carmen Bianca Bakker <carmen@carmenbianca.eu>
Co-authored-by: Carmen Bianca Bakker <carmen@carmenbianca.eu>
Co-authored-by: Pierre Verkest <pierreverkest84@gmail.com>
Currently translated at 100.0% (4 of 4 strings) Translation: pos-12.0/pos-12.0-pos_pricelist_technical Translate-URL: https://translation.odoo-community.org/projects/pos-12-0/pos-12-0-pos_pricelist_technical/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: pos-12.0/pos-12.0-pos_pricelist_technical Translate-URL: https://translation.odoo-community.org/projects/pos-12-0/pos-12-0-pos_pricelist_technical/
cd913cd
to
1d094c8
Compare
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at f800332. Thanks a lot for contributing to OCA. ❤️ |
Description
This module extends the functionality of Point of Sale to prevent technical pricelists that are used for the computation of other pricelists from being selected.
Configure
First configure the main module
sale_pricelist_technical
.Configure your pricelist on your pos config form.
Usage