Compare commits

...
Author SHA1 Message Date
gitea-actions c245f3b67a chore(gitops): deploy Docs Portal v1.0.276 [skip ci] 2026-09-20 20:42:06 +00:00
devops 149f743975 Merge branch 'docs/adr-0010-cierre-zoom-imagenes-manuales-2026-09-20'
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 37s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 9m16s
2026-09-20 20:32:18 +00:00
devops cc79d3b089 docs(adr-0010): cerrar bug de zoom con imagenes manuales, url() sin comillas 2026-09-20 20:31:58 +00:00
devops 5bcacdf09d chore: promote v1.0.275 to production [skip ci] 2026-09-20 20:22:40 +00:00
devops 3e5e18c14f chore: release v1.0.275 [skip ci] 2026-09-20 20:22:36 +00:00
devops e75a11d554 Merge branch 'fix/gallery-zoom-url-spaces-2026-09-20'
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (push) Successful in 39s
Build and Push Frontend / Construir y Subir Imagen (push) Successful in 13m30s
2026-09-20 20:07:58 +00:00
devops 0fd05dab2a fix(ecommerce): comillas en url() del zoom, rompia con espacios/parentesis en el nombre de archivo 2026-09-20 20:07:32 +00:00
gitea-actions 5fe18ce3d4 chore(gitops): deploy Docs Portal v1.0.274 [skip ci] 2026-09-20 19:25:20 +00:00
devops 2318e1da7d Merge branch 'docs/adr-0010-cierre-bugs-mobile-imagenes-2026-09-20'
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 31s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 6m18s
2026-09-20 19:18:22 +00:00
5 changed files with 46 additions and 16 deletions

No files matched your search

@@ -425,10 +425,10 @@ Resultados reales, uno por uno (código de cada prueba en
un dispositivo real. un dispositivo real.
21b. **Accesibilidad del checkout completo** por teclado (no solo el 21b. **Accesibilidad del checkout completo** por teclado (no solo el
buscador) — tab order y foco visible en los 3 pasos. buscador) — tab order y foco visible en los 3 pasos.
22. **Importación manual de imágenes no se reflejaba en el storefront** 22. **Importación manual de imágenes no se reflejaba en el storefront —
(reportado por el usuario 2026-09-20) — **causa raíz encontrada y RESUELTO 2026-09-20, con un giro real durante la verificación.**
arreglada en código (2026-09-20), pendiente confirmar con una Reportado por el usuario 2026-09-20. Causa raíz #1 (caché
edición real.** `medusaFetch()` en `lib/headless/providers/medusa.ts` multi-pod): `medusaFetch()` en `lib/headless/providers/medusa.ts`
usaba `next:{revalidate:60,tags:["medusa-products"]}`, caché en usaba `next:{revalidate:60,tags:["medusa-products"]}`, caché en
memoria por-pod que sobrevivía a `force-dynamic`. Con memoria por-pod que sobrevivía a `force-dynamic`. Con
`frontend-deploy` en 2 réplicas sin cacheHandler compartido, el POST `frontend-deploy` en 2 réplicas sin cacheHandler compartido, el POST
@@ -437,11 +437,28 @@ Resultados reales, uno por uno (código de cada prueba en
TTL expirara solo. Una edición puntual + revisar al toque tenía TTL expirara solo. Una edición puntual + revisar al toque tenía
~50% de chance de pegarle al pod stale; una importación CSV (mucho ~50% de chance de pegarle al pod stale; una importación CSV (mucho
más lenta) le daba tiempo de sobra al TTL para expirar en ambos. más lenta) le daba tiempo de sobra al TTL para expirar en ambos.
**Fix:** se sacó el `next.revalidate`/`tags` del fetch — ya en **Fix #1:** se sacó el `next.revalidate`/`tags` del fetch.
producción. Falta la prueba con una edición manual real (vía
`temp-admin`/`ari-shopping-tienda` en memoria) para confirmar que Verificado en vivo por el usuario con una edición real (agregó
el síntoma original desapareció, no solo que la causa técnica tiene varias fotos manuales al producto `minnie-y-mickey-mouse`): las
sentido. fotos sí se reflejaron al toque, confirmando el fix #1 — pero
apareció un segundo síntoma real, no relacionado: "el zoom
interactivo no se visualiza" en las fotos subidas a mano, aunque sí
en las de CSV. **Causa raíz #2** (`components/product/product-gallery.tsx`):
el panel/overlay de zoom arma `backgroundImage: \`url(${selected})\``
sin comillas — un nombre de archivo con espacio y paréntesis (ej.
`descarga (4)-xxx.png`, el patrón por defecto de Chrome al descargar
una imagen sin renombrarla, exactamente como quedan las subidas
manuales, a diferencia de las de CSV que llegan con nombres
saneados) rompe la sintaxis de `url()` sin comillas — el navegador
descarta el valor completo en silencio (`backgroundImage: none`).
La lupa (recuadro) sí aparecía porque no depende de la URL; solo el
panel/overlay quedaba en blanco. **Fix #2:** comillas alrededor de
la URL en las dos ocurrencias del patrón. Verificado en vivo contra
las 9 fotos reales del producto (2 de CSV + 7 manuales con
espacio+paréntesis) con Playwright: `computed backgroundImage`
pasó de `none` a `url("...")` correcto en las 7 que fallaban, sin
tocar las 2 que ya andaban bien.
23. **En mobile (viewport real de 320px), la página se cortaba y 23. **En mobile (viewport real de 320px), la página se cortaba y
obligaba a scrollear horizontalmente — RESUELTO 2026-09-20.** obligaba a scrollear horizontalmente — RESUELTO 2026-09-20.**
Reportado por el usuario con captura real de `shop.cruzcloud.net`. Reportado por el usuario con captura real de `shop.cruzcloud.net`.
@@ -25,7 +25,7 @@ spec:
runAsNonRoot: false runAsNonRoot: false
containers: containers:
- name: app - name: app
image: gitea.cruzcloud.net/devops/docs-portal:v1.0.269 image: gitea.cruzcloud.net/devops/docs-portal:v1.0.276
# El base saca TODAS las capabilities. Probado en vivo y # El base saca TODAS las capabilities. Probado en vivo y
# revertido: con drop: ALL (aunque se agregue NET_BIND_SERVICE # revertido: con drop: ALL (aunque se agregue NET_BIND_SERVICE
# de vuelta) el propio entrypoint de nginx:1.27-alpine falla al # de vuelta) el propio entrypoint de nginx:1.27-alpine falla al
@@ -26,11 +26,11 @@ spec:
runAsUser: 1001 runAsUser: 1001
containers: containers:
- name: app - name: app
image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.273 image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.275
env: env:
# Ver comentario en workloads/ecommerce/frontend/patches/deployment.yaml # Ver comentario en workloads/ecommerce/frontend/patches/deployment.yaml
- name: APP_VERSION - name: APP_VERSION
value: v1.0.273 value: v1.0.275
- name: HEADLESS_PROVIDER - name: HEADLESS_PROVIDER
value: medusa value: medusa
# Backend COMPARTIDO con Producción a propósito (decisión del # Backend COMPARTIDO con Producción a propósito (decisión del
@@ -179,7 +179,18 @@ export function ProductGallery({ images, name }: { images: string[]; name: strin
className="pointer-events-none absolute inset-0 bg-slate-50 transition-opacity duration-150 ease-out" className="pointer-events-none absolute inset-0 bg-slate-50 transition-opacity duration-150 ease-out"
style={{ style={{
opacity: hoverZoom && !hoverZoom.showPanel ? 1 : 0, opacity: hoverZoom && !hoverZoom.showPanel ? 1 : 0,
backgroundImage: `url(${selected})`, // Comillas dentro de url() a proposito: sin ellas, un
// espacio o parentesis literal en el nombre de archivo
// (ej. "descarga (4)-xxx.png", el patron por defecto de
// Chrome al descargar una imagen sin renombrarla -- asi
// quedan las subidas manuales, a diferencia de las
// importadas por CSV) rompe la sintaxis de url() sin
// comillas -- el navegador descarta el valor completo en
// silencio (backgroundImage queda "none"). Confirmado en
// vivo 2026-09-20: reportado por el usuario como "el zoom
// interactivo no se visualiza" solo en fotos subidas a
// mano por el admin, nunca en las de CSV.
backgroundImage: `url("${selected}")`,
backgroundRepeat: "no-repeat", backgroundRepeat: "no-repeat",
backgroundSize: `${HOVER_ZOOM_SCALE * 100}%`, backgroundSize: `${HOVER_ZOOM_SCALE * 100}%`,
backgroundPosition: hoverZoom ? `${hoverZoom.bgX}% ${hoverZoom.bgY}%` : "center", backgroundPosition: hoverZoom ? `${hoverZoom.bgX}% ${hoverZoom.bgY}%` : "center",
@@ -217,7 +228,9 @@ export function ProductGallery({ images, name }: { images: string[]; name: strin
top: hoverZoom.panelTop, top: hoverZoom.panelTop,
width: hoverZoom.panelSize, width: hoverZoom.panelSize,
height: hoverZoom.panelSize, height: hoverZoom.panelSize,
backgroundImage: `url(${selected})`, // Ver el comentario en el overlay in-place de arriba -- mismo
// fix, mismo motivo (comillas necesarias dentro de url()).
backgroundImage: `url("${selected}")`,
backgroundRepeat: "no-repeat", backgroundRepeat: "no-repeat",
backgroundSize: `${HOVER_ZOOM_SCALE * 100}%`, backgroundSize: `${HOVER_ZOOM_SCALE * 100}%`,
backgroundPosition: `${hoverZoom.bgX}% ${hoverZoom.bgY}%`, backgroundPosition: `${hoverZoom.bgX}% ${hoverZoom.bgY}%`,
@@ -25,14 +25,14 @@ spec:
runAsUser: 1001 runAsUser: 1001
containers: containers:
- name: app - name: app
image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.273 image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.275
env: env:
# La actualiza el mismo paso del CD (.gitea/workflows/build.yaml) # La actualiza el mismo paso del CD (.gitea/workflows/build.yaml)
# que bumpea el tag de la imagen -- el footer la lee # que bumpea el tag de la imagen -- el footer la lee
# (process.env.APP_VERSION) para mostrar la versión real # (process.env.APP_VERSION) para mostrar la versión real
# desplegada en vez de un "v4.1" fijo. # desplegada en vez de un "v4.1" fijo.
- name: APP_VERSION - name: APP_VERSION
value: v1.0.273 value: v1.0.275
- name: HEADLESS_PROVIDER - name: HEADLESS_PROVIDER
value: medusa value: medusa
- name: MEDUSA_BACKEND_URL - name: MEDUSA_BACKEND_URL