-
Notifications
You must be signed in to change notification settings - Fork 5
[IMP] pos_membership_extension: consider membership in shopping cart #39
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
[IMP] pos_membership_extension: consider membership in shopping cart #39
Conversation
…tail of the taxes. (Tax amount and base amount).
… the proxy + raise an error when its not properly configured
Currently translated at 75.0% (9 of 12 strings) Translation: pos-16.0/pos-16.0-pos_daily_sales_reports_category_only Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_daily_sales_reports_category_only/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: pos-16.0/pos-16.0-pos_receipt_usability Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_receipt_usability/
Currently translated at 100.0% (2 of 2 strings) Translation: pos-16.0/pos-16.0-pos_payment_usability Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_payment_usability/it/
Currently translated at 100.0% (5 of 5 strings) Translation: pos-16.0/pos-16.0-pos_receipt_vat_detail Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_receipt_vat_detail/it/
Currently translated at 100.0% (2 of 2 strings) Translation: pos-16.0/pos-16.0-pos_cash_control_override Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_cash_control_override/it/
Currently translated at 100.0% (12 of 12 strings) Translation: pos-16.0/pos-16.0-pos_container_deposit Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_container_deposit/it/
…..' no-empty-function
… no-useless-constructor
…de no-inline-comments
…house_id) of pos.config() have the same label: Warehouse. [Modules: pos_stock_available_online and point_of_sale]
…class(*classes) function to filter elements by their classes
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: pos-16.0/pos-16.0-pos_stock_available_online Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_stock_available_online/
…rite on many items. (for exemple many quants)
Currently translated at 100.0% (19 of 19 strings) Translation: pos-16.0/pos-16.0-pos_stock_available_online Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_stock_available_online/it/
Recompute `sale.order` taxes, as received ones are the ones from product, without fiscal position mapping applied.
Hi @legalsylvain, |
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.
LGTM. Please do a PR against https://github.com/OCA/pos.
Please, edit the readme (by modifying files in readme
folder) to explain your changes.
The sale of a product requiring membership is allowed if the membership product is in the shopping cart, even if the customer is not yet a member.
06c69e9
to
9b42feb
Compare
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## 16.0 #39 +/- ##
===========================================
- Coverage 89.56% 69.69% -19.88%
===========================================
Files 141 322 +181
Lines 1016 2445 +1429
Branches 88 139 +51
===========================================
+ Hits 910 1704 +794
- Misses 84 727 +643
+ Partials 22 14 -8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
See : OCA#1341 |
Allow the sale of a product requiring membership if the membership product is in the shopping cart, even if the customer is not yet a member
Odoo task : T13569