contentoh-components-library 21.6.33 → 21.6.34

package/.claude/launch.json

@@ -0,0 +1,15 @@
+{
+  "version": "0.0.1",
+  "configurations": [
+    {
+      "name": "creators",
+      "runtimeExecutable": "sh",
+      "runtimeArgs": [
+        "-c",
+        "cd /Users/macbook/contentoh/content-creation-frontend && export PATH=/Users/macbook/.nvm/versions/node/v16.20.2/bin:$PATH && exec npm start"
+      ],
+      "port": 3000,
+      "autoPort": false
+    }
+  ]
+}

package/dist/components/organisms/LinkVariantsModal/styles.js

@@ -82,7 +82,7 @@ var ImageGrid = _styledComponents.default.div(_templateObject12 || (_templateObj
 
 exports.ImageGrid = ImageGrid;
 
-var Direction = _styledComponents.default.div(_templateObject13 || (_templateObject13 = (0, _taggedTemplateLiteral2.default)(["\n  display: flex;\n  align-items: stretch;\n  gap: 10px;\n  margin-bottom: 12px;\n\n  .lado {\n    flex: 1 1 0;\n    min-width: 0;\n    padding: 10px 12px;\n    border: 1px solid ", ";\n    border-radius: 6px;\n    background-color: ", ";\n  }\n\n  /* El padre es el que manda en esta pantalla, y se marca como tal. */\n  .lado:first-of-type {\n    border-color: rgba(", ", 0.45);\n    background-color: rgba(", ", 0.05);\n  }\n\n  .rol {\n    display: block;\n    font-family: ", ";\n    font-size: 11px;\n    text-transform: uppercase;\n    letter-spacing: 0.4px;\n    color: ", ";\n  }\n\n  strong {\n    display: block;\n    margin-top: 2px;\n    font-family: ", ";\n    font-size: 13px;\n    color: ", ";\n    overflow: hidden;\n    text-overflow: ellipsis;\n    white-space: nowrap;\n  }\n\n  /* Cu\xE1l de los dos es la ficha abierta: sin esto, en modo \"este producto es\n     hijo de otro\" no se sabe de qu\xE9 lado est\xE1 uno. */\n  .actual {\n    display: block;\n    margin-top: 2px;\n    font-size: 11px;\n    font-style: italic;\n    color: ", ";\n  }\n\n  .flecha {\n    align-self: center;\n    flex-shrink: 0;\n    font-size: 20px;\n    line-height: 1;\n    color: ", ";\n  }\n"])), _variables.GlobalColors.s2, _variables.GlobalColors.white, PINK_RGB, PINK_RGB, _variables.FontFamily.Raleway_600, PINK, _variables.FontFamily.Raleway_600, _variables.GlobalColors.deep_gray, _variables.GlobalColors.gray, PINK);
+var Direction = _styledComponents.default.div(_templateObject13 || (_templateObject13 = (0, _taggedTemplateLiteral2.default)(["\n  display: flex;\n  align-items: stretch;\n  gap: 10px;\n  margin-bottom: 12px;\n\n  .lado {\n    flex: 1 1 0;\n    min-width: 0;\n    padding: 10px 12px;\n    border: 1px solid ", ";\n    border-radius: 6px;\n    background-color: ", ";\n  }\n\n  /* El padre es el que manda en esta pantalla, y se marca como tal. */\n  .lado:first-of-type {\n    border-color: rgba(", ", 0.45);\n    background-color: rgba(", ", 0.05);\n  }\n\n  .rol {\n    display: block;\n    font-family: ", ";\n    font-size: 11px;\n    text-transform: uppercase;\n    letter-spacing: 0.4px;\n    color: ", ";\n  }\n\n  strong {\n    display: block;\n    margin-top: 2px;\n    font-family: ", ";\n    font-size: 13px;\n    color: ", ";\n    overflow: hidden;\n    text-overflow: ellipsis;\n    white-space: nowrap;\n  }\n\n  /* Cu\xE1l de los dos es la ficha abierta: sin esto, al relacionar un padre no se\n     sabe de qu\xE9 lado est\xE1 uno. */\n  .actual {\n    display: block;\n    margin-top: 2px;\n    font-size: 11px;\n    font-style: italic;\n    color: ", ";\n  }\n\n  .flecha {\n    align-self: center;\n    flex-shrink: 0;\n    font-size: 20px;\n    line-height: 1;\n    color: ", ";\n  }\n"])), _variables.GlobalColors.s2, _variables.GlobalColors.white, PINK_RGB, PINK_RGB, _variables.FontFamily.Raleway_600, PINK, _variables.FontFamily.Raleway_600, _variables.GlobalColors.deep_gray, _variables.GlobalColors.gray, PINK);
 
 exports.Direction = Direction;
 

package/dist/components/pages/ProviderProductEdition/index.js

@@ -315,7 +315,20 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
     };
 
     setProductData();
-  }, []); // Setter de los servicios y porcentages del producto
+  }, []);
+  /*
+    Contador que fuerza a releer los servicios tras ligar.
+     El efecto de abajo depende de estatus, versión y cadena activa, y ligar no
+    cambia ninguno: cambia la HERENCIA de cada campo en el servidor. Al planchar,
+    lo que era propio pasa a heredado, y sin releer los chips siguen diciendo lo
+    de antes hasta que alguien cierra y reabre el producto.
+  */
+
+  var _useState31 = (0, _react.useState)(0),
+      _useState32 = (0, _slicedToArray2.default)(_useState31, 2),
+      linkNonce = _useState32[0],
+      setLinkNonce = _useState32[1]; // Setter de los servicios y porcentages del producto
+
 
   (0, _react.useEffect)(function () {
     if (!state.product || !state.active_retailer) return;
@@ -481,7 +494,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
     }();
 
     fetchData();
-  }, [(_state$product2 = state.product) === null || _state$product2 === void 0 ? void 0 : _state$product2.datasheet_status, (_state$product3 = state.product) === null || _state$product3 === void 0 ? void 0 : _state$product3.description_status, (_state$product4 = state.product) === null || _state$product4 === void 0 ? void 0 : _state$product4.images_status, (_state$product5 = state.product) === null || _state$product5 === void 0 ? void 0 : _state$product5.status, (_state$product6 = state.product) === null || _state$product6 === void 0 ? void 0 : _state$product6.percentages, (_state$product7 = state.product) === null || _state$product7 === void 0 ? void 0 : _state$product7.version, state.active_retailer, token]); // Actualizador de campos del ui para que coincidan con el retailer activo
+  }, [(_state$product2 = state.product) === null || _state$product2 === void 0 ? void 0 : _state$product2.datasheet_status, (_state$product3 = state.product) === null || _state$product3 === void 0 ? void 0 : _state$product3.description_status, (_state$product4 = state.product) === null || _state$product4 === void 0 ? void 0 : _state$product4.images_status, (_state$product5 = state.product) === null || _state$product5 === void 0 ? void 0 : _state$product5.status, (_state$product6 = state.product) === null || _state$product6 === void 0 ? void 0 : _state$product6.percentages, (_state$product7 = state.product) === null || _state$product7 === void 0 ? void 0 : _state$product7.version, state.active_retailer, token, linkNonce]); // Actualizador de campos del ui para que coincidan con el retailer activo
 
   (0, _react.useEffect)(function () {
     if (!state.active_retailer) return;
@@ -1187,7 +1200,11 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
       payload: updatedProduct
     }); // La lista cacheada quedó vieja: se descarta para que vuelva a pedirse.
 
-    setFamily(null); // Y en sessionStorage, que es de donde sale el producto al recargar. Sin
+    setFamily(null); // Y los servicios, para que los chips reflejen la herencia recién creada.
+
+    setLinkNonce(function (n) {
+      return n + 1;
+    }); // Y en sessionStorage, que es de donde sale el producto al recargar. Sin
     // esto el arreglo dura hasta el siguiente F5 y vuelve el contador viejo,
     // que es justo el síntoma que se reportó.
 

package/dist/components/pages/RetailerProductEdition/index.js

@@ -1233,8 +1233,9 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
   */
 
   /*
-    Refresca tras ligar, sin recargar: el producto sale de sessionStorage, así que
-    un refresh devolvería el mismo objeto y el contador seguiría igual.
+    Refresca tras ligar, sin recargar la página: el producto sale de
+    sessionStorage, así que un F5 devolvería el mismo objeto y el contador
+    seguiría igual.
   */
 
   var handleOnLinkedVariants = function handleOnLinkedVariants(result) {
@@ -1251,13 +1252,34 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
       type: "SET_PRODUCT",
       payload: updatedProduct
     });
-    setFamily(null);
 
     try {
       sessionStorage.setItem("productSelected", JSON.stringify(updatedProduct));
     } catch (error) {
       console.error("linkVariants/sessionStorage: ", error);
     }
+    /*
+      Volver a preguntar, no vaciar.
+       Antes esto hacía `setFamily(null)` para invalidar la caché, y en esta
+      pantalla eso tenía el efecto contrario al buscado: el bloque de parentesco
+      se decide con `esHijoDeOtro`, que sale de `family`, no del producto. Con
+      `family` en null el bloque desaparecía, así que después de ligar la
+      pantalla se quedaba SIN ninguna señal de que el producto ya tenía padre
+      —justo lo que el usuario acababa de hacer—.
+    */
+
+
+    loadFamily(true);
+    /*
+      Y se recargan los servicios, porque ligar cambia la herencia de cada campo
+      en el servidor: al planchar, lo que era propio pasa a heredado y los chips
+      tienen que decirlo. Sin esto la ficha sigue mostrando el estado anterior
+      hasta que alguien cierra y vuelve a abrir el producto.
+       Sin `enableLoading` para no parpadear el esqueleto encima de una pantalla
+      que ya está pintada.
+    */
+
+    loadData(false);
   };
   /*
     Acción "Volver a heredar" de los chips. Repinta en sitio con lo que devuelve

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "contentoh-components-library",
-  "version": "21.6.33",
+  "version": "21.6.34",
   "dependencies": {
     "@aws-amplify/auth": "^4.5.3",
     "@aws-amplify/datastore": "^3.11.0",

package/src/components/pages/ProviderProductEdition/index.js

@@ -207,6 +207,16 @@ const ProviderProductEditionView = ({
     setProductData();
   }, []);
 
+  /*
+    Contador que fuerza a releer los servicios tras ligar.
+
+    El efecto de abajo depende de estatus, versión y cadena activa, y ligar no
+    cambia ninguno: cambia la HERENCIA de cada campo en el servidor. Al planchar,
+    lo que era propio pasa a heredado, y sin releer los chips siguen diciendo lo
+    de antes hasta que alguien cierra y reabre el producto.
+  */
+  const [linkNonce, setLinkNonce] = useState(0);
+
   // Setter de los servicios y porcentages del producto
   useEffect(() => {
     if (!state.product || !state.active_retailer) return;
@@ -363,6 +373,7 @@ const ProviderProductEditionView = ({
     state.product?.version,
     state.active_retailer,
     token,
+    linkNonce,
   ]);
 
   // Actualizador de campos del ui para que coincidan con el retailer activo
@@ -941,6 +952,8 @@ const ProviderProductEditionView = ({
     dispatch({ type: "SET_PRODUCT", payload: updatedProduct });
     // La lista cacheada quedó vieja: se descarta para que vuelva a pedirse.
     setFamily(null);
+    // Y los servicios, para que los chips reflejen la herencia recién creada.
+    setLinkNonce((n) => n + 1);
 
     // Y en sessionStorage, que es de donde sale el producto al recargar. Sin
     // esto el arreglo dura hasta el siguiente F5 y vuelve el contador viejo,

package/src/components/pages/RetailerProductEdition/index.js

@@ -849,8 +849,9 @@ const RetailerProductEditionView = ({
   */
 
   /*
-    Refresca tras ligar, sin recargar: el producto sale de sessionStorage, así que
-    un refresh devolvería el mismo objeto y el contador seguiría igual.
+    Refresca tras ligar, sin recargar la página: el producto sale de
+    sessionStorage, así que un F5 devolvería el mismo objeto y el contador
+    seguiría igual.
   */
   const handleOnLinkedVariants = (result) => {
     if (!result) return;
@@ -869,13 +870,35 @@ const RetailerProductEditionView = ({
           };
 
     dispatch({ type: "SET_PRODUCT", payload: updatedProduct });
-    setFamily(null);
 
     try {
       sessionStorage.setItem("productSelected", JSON.stringify(updatedProduct));
     } catch (error) {
       console.error("linkVariants/sessionStorage: ", error);
     }
+
+    /*
+      Volver a preguntar, no vaciar.
+
+      Antes esto hacía `setFamily(null)` para invalidar la caché, y en esta
+      pantalla eso tenía el efecto contrario al buscado: el bloque de parentesco
+      se decide con `esHijoDeOtro`, que sale de `family`, no del producto. Con
+      `family` en null el bloque desaparecía, así que después de ligar la
+      pantalla se quedaba SIN ninguna señal de que el producto ya tenía padre
+      —justo lo que el usuario acababa de hacer—.
+    */
+    loadFamily(true);
+
+    /*
+      Y se recargan los servicios, porque ligar cambia la herencia de cada campo
+      en el servidor: al planchar, lo que era propio pasa a heredado y los chips
+      tienen que decirlo. Sin esto la ficha sigue mostrando el estado anterior
+      hasta que alguien cierra y vuelve a abrir el producto.
+
+      Sin `enableLoading` para no parpadear el esqueleto encima de una pantalla
+      que ya está pintada.
+    */
+    loadData(false);
   };
 
   /*