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_configurator_base: Only prevent direct sol edition if product was added by the sale configurator interface #42

Open
wants to merge 1 commit into
base: 14.0
Choose a base branch
from

Conversation

paradoxxxzero
Copy link
Contributor

No description provided.

@paradoxxxzero paradoxxxzero force-pushed the 14.0-sale_configurator_allow_direct_edit_on_non_configured_sol branch from cea4e39 to eeb4903 Compare April 4, 2022 17:15
@paradoxxxzero paradoxxxzero force-pushed the 14.0-sale_configurator_allow_direct_edit_on_non_configured_sol branch from eeb4903 to 8eadde0 Compare September 20, 2022 07:46
@paradoxxxzero paradoxxxzero force-pushed the 14.0-sale_configurator_allow_direct_edit_on_non_configured_sol branch from 8eadde0 to 6676be7 Compare February 8, 2023 10:19
@codecov
Copy link

codecov bot commented Feb 8, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.46%. Comparing base (b1bc0ef) to head (0a5093b).

Additional details and impacted files
@@            Coverage Diff             @@
##             14.0      #42      +/-   ##
==========================================
+ Coverage   82.44%   82.46%   +0.02%     
==========================================
  Files          47       47              
  Lines         826      827       +1     
  Branches      154      154              
==========================================
+ Hits          681      682       +1     
  Misses        127      127              
  Partials       18       18              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

<button
title="This product was not added as configurable"
icon="fa-ban"
attrs="{'invisible': ['|', ('state', 'not in', ('draft', 'sent')), '|', ('is_configurable', '=', False), ('is_configured', '=', True)]}"
Copy link
Member

Choose a reason for hiding this comment

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

If configurable product was added without using configuring interface, we cant edit it to add option ?

…uct was added by the sale configurator interface
@paradoxxxzero paradoxxxzero force-pushed the 14.0-sale_configurator_allow_direct_edit_on_non_configured_sol branch from 6676be7 to 0a5093b Compare March 4, 2024 12:20
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.

3 participants