Oliver/loreal-sla-calculator
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix: Translation/Syndication toggles now override base stage matrix

Browse source 
The base stage matrix for brief types like country_pull_adaptation
includes Translation stages by default. In the Brief Advisor, the
user's toggles should be authoritative — if Translation is OFF, those
stages must be removed regardless of the base matrix.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Alessandro Benedetti Admin 2026-03-16 14:01:09 +00:00
parent ac9a3f057b
commit d98dadec8f
1 changed files with 5 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
market-script.js
View file

@ -224,15 +224,11 @@ function calculateAndRender() {
if (sCfg && sCfg.alwaysActive) matrix[i] = true;
});
// Client toggles can activate translation/syndication even if the base
// brief type doesn't include them
if (needsTranslation) {
matrix[3] = true; // Translation (Salsify PDP)
matrix[4] = true; // Translation (Asset)
}
if (needsSyndication && !matrix[7]) {
matrix[7] = true; // Syndication
}
// In the Brief Advisor the user's toggles are the source of truth
// for translation and syndication — override the base matrix
matrix[3] = needsTranslation; // Translation (Salsify PDP)
matrix[4] = needsTranslation; // Translation (Asset)
matrix[7] = needsSyndication; // Syndication
// Combined feedback days from production toggles (static/video/html)
// Uses the max of the user-supplied feedback values for production stages