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

[IMP] website_product_configurator: Show UoM for custom values #113

Open
wants to merge 261 commits into
base: 16.0
Choose a base branch
from

Conversation

SirAionTech
Copy link

Based on #103.

@SirAionTech SirAionTech marked this pull request as ready for review February 22, 2024 11:21
matt454357 and others added 29 commits February 22, 2024 15:28
Completes the following TODOs:
* Refactor so multiple values can be checked at once
  also a better method for building the domain using the logical
  operators is required
* Enable the usage of OR operators between domain lines

Adds the following TODOs:
* Enable the usage of OR operators between implied_ids
* Add implied_ids sequence field to enforce order of operations
…igurator_mrp : correct method-name, claas-name, arguments
…, replace request.website.render with request.render
…production_lots, fix singleton error(product.config.step.line)
… website_product_configurator_mrp to uninstallable list
bizzappdev and others added 29 commits February 22, 2024 15:29
Fix hasClass('.cfg-radio') -> With the DOT, the class is never founded
Use getAttribute("name") instead of attribute[0].name because it crash with radio-button div
…sion and also get value from get_attribute_value_extra_prices
@SirAionTech SirAionTech force-pushed the 16.0-imp-website_product_configurator-show_uom branch from 697b52b to d8b3d49 Compare February 22, 2024 14:34
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.

None yet

7 participants