Currently, adjustments are reset during the refresh process. In #2805343: Adjustments added by the UI are removed in order refresh. we have a quick hack proposed to preserve adjustments added in the user interface.

Ideally, we need a way to "lock" adjustments that prevent them from being removed in the refresh process. This will force them to be explicitly removed, and allow other implementations to provide persisting adjustments.

Comments

mglaman created an issue. See original summary.

dravenk’s picture

dravenk
He/Him
Primary language English
CreditAttribution: dravenk as a volunteer and at Sparkpad, Google Summer of Code commented
Assigned: Unassigned » dravenk
dravenk’s picture

dravenk
He/Him
Primary language English
CreditAttribution: dravenk as a volunteer and at Sparkpad, Google Summer of Code commented

:( I will try to do with it. but, what to start? I would need more details ...

dravenk’s picture

dravenk
He/Him
Primary language English
CreditAttribution: dravenk as a volunteer and at Sparkpad, Google Summer of Code commented

:( I will try to do with it. but, what to start? I would need more details ...

skyredwang’s picture

skyredwang
Location Shanghai
CreditAttribution: skyredwang at INsReady for Sparkpad commented

bojanz provided more info on IRC:

<bojanz> skyred: we need a $locked property, the same way we have an $included property. And the code that clears "everything except Custom" needs to clear "everything except locked"
9:40 PM <bojanz> skyred: and all UI created adjustments should be locked
bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented

Let's split the UI part to #2840134: Rewrite AdjustmentDefaultWidget to use #ajax and just handle the API side here. That way we can get something in quicker.

  • bojanz committed e76e1c2 on 8.x-2.x 
    Issue #2805355 by bojanz: Add the ability to "Lock" an Adjustment so it...
bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Active » Fixed

Committed the API side.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.