Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
0fd05dab2a | ||
|
|
2318e1da7d | ||
|
|
b6b2f15c12 | ||
|
|
7658b9879a | ||
|
|
a1dc145528 | ||
|
|
4f5a715812 | ||
|
|
5ce9c07829 | ||
|
|
428ad83e5b | ||
|
|
80862bef5b | ||
|
|
777e47ce16 | ||
|
|
167ae4057a | ||
|
|
51300c234a | ||
|
|
0c0efe3727 | ||
|
|
e4dafe9017 | ||
|
|
fed5dfc1c1 | ||
|
|
5773154db9 | ||
|
|
553b1c7a32 | ||
|
|
d9c39ba2a1 | ||
|
|
164223e844 | ||
|
|
ee327b420c | ||
|
|
17348e6285 | ||
|
|
6c6303cf8e | ||
|
|
410975f972 | ||
|
|
291db05a5e | ||
|
|
73ff01c84a | ||
|
|
5933279713 | ||
|
|
a77a6f2fd0 | ||
|
|
e3c27de736 | ||
|
|
cc7ddddf58 | ||
|
|
7dd2c139cb | ||
|
|
fe4ed8005a | ||
|
|
009d5c63df | ||
|
|
4148f14128 | ||
|
|
b3b7ecc77a | ||
|
|
d3e0d78314 | ||
|
|
5c79cc3345 | ||
|
|
972b11772a | ||
|
|
90176e36fc | ||
|
|
5b8bac5ef1 | ||
|
|
545062c3ef | ||
|
|
4009fde072 | ||
|
|
0723b43c54 | ||
|
|
58a1fd1597 | ||
|
|
8889fd9ea2 | ||
|
|
567ee30e1f | ||
|
|
cd72962fa4 | ||
|
|
af9576f45d | ||
|
|
1c9693e573 | ||
|
|
f92e8e6b36 | ||
|
|
ccaddf546c | ||
|
|
8d8546fd94 | ||
|
|
57acdb0ed3 | ||
|
|
5a2d0c061c | ||
|
|
c3f3857a92 | ||
|
|
9b545dd850 | ||
|
|
401f43a136 | ||
|
|
b23f19ac6b | ||
|
|
9d4553f3a4 | ||
|
|
d605417452 | ||
|
|
706732878f | ||
|
|
c222a6fed6 | ||
|
|
8d16f2b854 | ||
|
|
f0b1d7b745 | ||
|
|
9ab779e781 | ||
|
|
2fa404fe7b | ||
|
|
69cc254051 | ||
|
|
76b7d827c5 | ||
|
|
3efab94e2b | ||
|
|
71c8752eeb | ||
|
|
cda25dea2e | ||
|
|
392a832109 | ||
|
|
a069807960 | ||
|
|
96b08eda01 | ||
|
|
e6c3e0ed8d | ||
|
|
35fdea5df5 | ||
|
|
8d44f2e771 | ||
|
|
ef6dc1f6a8 | ||
|
|
352762b9d2 | ||
|
|
58cc93bb2f | ||
|
|
5eaa94818f | ||
|
|
66a81ad9a3 | ||
|
|
39e1bf61d3 | ||
|
|
b5a1f70d1d |
No files matched your search
@@ -126,8 +126,20 @@ jobs:
|
||||
|
||||
env:
|
||||
APP_DIR: workloads/ecommerce
|
||||
MANIFEST_FILE: workloads/ecommerce/frontend/patches/deployment.yaml
|
||||
KUSTOMIZE_DIR: workloads/ecommerce/frontend
|
||||
# Cambio 2026-09-14: todo push a main promueve a STAGING
|
||||
# automaticamente (workloads/ecommerce-staging), no a Produccion
|
||||
# directo -- decision del usuario para dejar de probar cambios en
|
||||
# vivo contra Prod.
|
||||
# Cambio 2026-09-20: la promocion de Staging a Produccion (antes
|
||||
# scripts/promote-to-production.sh, manual con confirmacion
|
||||
# interactiva) ahora corre sola en este mismo job -- ver el step
|
||||
# "Promover a Produccion (CD)" mas abajo, justo despues del de
|
||||
# Staging. Decision explicita del usuario: cero gate humano entre
|
||||
# Staging y Produccion. El script manual queda solo como
|
||||
# herramienta de rollback/excepcion, ya no es el camino normal.
|
||||
MANIFEST_FILE: workloads/ecommerce-staging/frontend/patches/deployment.yaml
|
||||
PROD_MANIFEST_FILE: workloads/ecommerce/frontend/patches/deployment.yaml
|
||||
KUSTOMIZE_DIR: workloads/ecommerce-staging/frontend
|
||||
IMAGE_NAME: gitea.cruzcloud.net/devops/ecommerce-frontend
|
||||
|
||||
steps:
|
||||
@@ -579,6 +591,13 @@ jobs:
|
||||
|
||||
sed -i -E "s|(image: ${IMAGE_NAME}:).*|\1${VERSION}|g" "${MANIFEST_FILE}"
|
||||
|
||||
# El footer (workloads/ecommerce/components/layout/footer.tsx) lee
|
||||
# process.env.APP_VERSION para mostrar la versión real desplegada
|
||||
# -- este bump la mantiene igual al tag de la imagen de arriba.
|
||||
# -0777 (slurp): el nombre y el valor están en líneas distintas del
|
||||
# YAML, un sed de una sola línea no puede verlos juntos.
|
||||
perl -0777 -pi -e "s/(- name: APP_VERSION\n\s*value: ).*/\${1}${VERSION}/" "${MANIFEST_FILE}"
|
||||
|
||||
git add "${MANIFEST_FILE}"
|
||||
|
||||
if git diff --cached --quiet; then
|
||||
@@ -592,6 +611,48 @@ jobs:
|
||||
# sobrescribir commits ajenos o promocionar una versión antigua.
|
||||
git push origin HEAD:main
|
||||
|
||||
# Automatiza lo que antes era scripts/promote-to-production.sh (paso
|
||||
# manual con confirmación interactiva, read -rp). Decisión del
|
||||
# usuario 2026-09-20: cero pasos manuales entre Staging y
|
||||
# Producción — la misma imagen que acaba de promocionar a Staging
|
||||
# arriba se promueve sola a Producción, sin gate humano en el
|
||||
# medio. Mismo sed/perl que el script manual (mismo MANIFEST, otro
|
||||
# archivo), corriendo como un segundo commit separado justo
|
||||
# después del de Staging — no se pisan en un solo commit porque
|
||||
# Staging y Producción son releases conceptualmente distintos
|
||||
# (permite revertir uno sin tocar el otro).
|
||||
- name: Promover a Producción (CD)
|
||||
if: steps.promotion.outputs.promote == 'true'
|
||||
shell: bash
|
||||
run: |
|
||||
set -euo pipefail
|
||||
|
||||
VERSION="${{ steps.vars.outputs.VERSION }}"
|
||||
|
||||
git config user.name "Cristian Felipe"
|
||||
git config user.email "[email protected]"
|
||||
|
||||
# Re-fetch: el step de Staging de arriba ya pusheó un commit a
|
||||
# main en este mismo job -- sin este refresh, este checkout
|
||||
# local queda un commit atrás y el push de abajo se rechaza.
|
||||
git fetch origin main
|
||||
git checkout -B main origin/main
|
||||
|
||||
sed -i -E "s|(image: ${IMAGE_NAME}:).*|\1${VERSION}|g" "${PROD_MANIFEST_FILE}"
|
||||
|
||||
perl -0777 -pi -e "s/(- name: APP_VERSION\n\s*value: ).*/\${1}${VERSION}/" "${PROD_MANIFEST_FILE}"
|
||||
|
||||
git add "${PROD_MANIFEST_FILE}"
|
||||
|
||||
if git diff --cached --quiet; then
|
||||
echo "Producción ya está en ${VERSION}."
|
||||
exit 0
|
||||
fi
|
||||
|
||||
git commit -m "chore: promote ${VERSION} to production [skip ci]"
|
||||
|
||||
git push origin HEAD:main
|
||||
|
||||
- name: Resumen del pipeline
|
||||
if: always()
|
||||
shell: bash
|
||||
|
||||
@@ -0,0 +1,26 @@
|
||||
apiVersion: argoproj.io/v1alpha1
|
||||
kind: Application
|
||||
|
||||
metadata:
|
||||
name: ecommerce-staging-app
|
||||
namespace: argocd
|
||||
|
||||
spec:
|
||||
project: default
|
||||
|
||||
source:
|
||||
repoURL: https://gitea.cruzcloud.net/devops/apps-registry.git
|
||||
targetRevision: main
|
||||
path: workloads/ecommerce-staging
|
||||
kustomize: {}
|
||||
|
||||
destination:
|
||||
server: https://kubernetes.default.svc
|
||||
namespace: ecommerce-staging
|
||||
|
||||
syncPolicy:
|
||||
automated:
|
||||
prune: true
|
||||
selfHeal: true
|
||||
syncOptions:
|
||||
- CreateNamespace=true
|
||||
@@ -0,0 +1,20 @@
|
||||
apiVersion: argoproj.io/v1alpha1
|
||||
kind: Application
|
||||
metadata:
|
||||
name: ecommerce-staging-governance
|
||||
namespace: argocd
|
||||
spec:
|
||||
project: default
|
||||
source:
|
||||
repoURL: https://gitea.cruzcloud.net/devops/platform-infra.git
|
||||
path: ecommerce-staging-governance
|
||||
targetRevision: main
|
||||
destination:
|
||||
server: https://kubernetes.default.svc
|
||||
namespace: ecommerce-staging
|
||||
syncPolicy:
|
||||
automated:
|
||||
prune: true
|
||||
selfHeal: true
|
||||
syncOptions:
|
||||
- CreateNamespace=true
|
||||
@@ -160,7 +160,7 @@ module.exports = defineConfig({
|
||||
options: {
|
||||
apiKey: required("ENVIOCLICK_API_KEY"),
|
||||
sandbox: process.env.ENVIOCLICK_SANDBOX !== "false",
|
||||
originDaneCode: process.env.ENVIOCLICK_ORIGIN_DANE_CODE || "11001000",
|
||||
originDaneCode: process.env.ENVIOCLICK_ORIGIN_DANE_CODE || "76001000",
|
||||
originAddress: required("ENVIOCLICK_ORIGIN_ADDRESS"),
|
||||
originCompany: process.env.ENVIOCLICK_ORIGIN_COMPANY || "ARI Shopping",
|
||||
originFirstName: process.env.ENVIOCLICK_ORIGIN_FIRST_NAME || "ARI",
|
||||
|
||||
@@ -38,11 +38,22 @@ import { csvRowsToObjects, parseCsv } from "../../../lib/csv";
|
||||
// generaba un producto duplicado con handle distinto (confirmado en
|
||||
// vivo con "Hello Kitty & Friends": misma fila re-subida para sumar
|
||||
// fotos, dos productos con el mismo titulo en la tienda). El match es
|
||||
// por titulo exacto (case-insensitive/trim), igual que categoria y
|
||||
// coleccion; en un match, las fotos nuevas se agregan a las que ya
|
||||
// tenia (no se reemplazan) y la descripcion se actualiza si la fila
|
||||
// trae una. No toca precio/cantidad/categoria/coleccion del producto
|
||||
// existente -- eso se edita a mano en el admin si hace falta.
|
||||
// por titulo normalizado (case/espacios/guiones-comillas tipograficas,
|
||||
// ver normalizeTitleForMatch), igual que categoria y coleccion; en un
|
||||
// match, las fotos nuevas se agregan a las que ya tenia (no se
|
||||
// reemplazan) y la descripcion se actualiza si la fila trae una. No
|
||||
// toca precio/cantidad/categoria/coleccion del producto existente --
|
||||
// eso se edita a mano en el admin si hace falta.
|
||||
//
|
||||
// El match era por titulo exacto (case-insensitive/trim) hasta que
|
||||
// volvio a duplicar "Toy Story 5 - Woody": la fila reimportada traia
|
||||
// un guion largo "–" (pegado desde Excel/Sheets, que autocorrige "-" a
|
||||
// en-dash) en vez del guion normal "-" del producto original, asi que
|
||||
// el string no calzaba y se creo un producto nuevo en vez de
|
||||
// actualizar el existente (confirmado en vivo 2026-09-19). Ademas de
|
||||
// guiones, se normalizan comillas tipograficas por la misma razon --
|
||||
// cualquier CSV pegado desde un editor de texto enriquecido puede
|
||||
// traerlas.
|
||||
//
|
||||
// Las fotos se descargan del origen y se re-alojan en el mismo
|
||||
// MinIO/S3 que ya usa el resto del catalogo (uploadFilesWorkflow ->
|
||||
@@ -88,6 +99,22 @@ type RowResult = {
|
||||
// mismos, ASCII-only, en vez de dejarselo al default de Medusa. El sufijo
|
||||
// corto evita colisiones entre titulos parecidos/repetidos sin necesidad
|
||||
// de revisar unicidad antes de crear.
|
||||
// Colapsa variantes de guion (-, en-dash, em-dash, minus) y comillas
|
||||
// tipograficas a su forma ASCII antes de comparar titulos, para que un
|
||||
// re-import pegado desde Excel/Sheets/Word (que autocorrigen esos
|
||||
// caracteres) siga matcheando el producto ya existente. Ver nota mas
|
||||
// arriba sobre "Toy Story 5 - Woody".
|
||||
function normalizeTitleForMatch(title: string): string {
|
||||
return title
|
||||
.normalize("NFKC")
|
||||
.replace(/[‐-―−]/g, "-")
|
||||
.replace(/[‘’‛]/g, "'")
|
||||
.replace(/[“”‟]/g, '"')
|
||||
.replace(/\s+/g, " ")
|
||||
.trim()
|
||||
.toLowerCase();
|
||||
}
|
||||
|
||||
function slugify(title: string, uniquePart: string): string {
|
||||
const base = title
|
||||
.normalize("NFD")
|
||||
@@ -190,13 +217,13 @@ export async function POST(req: MedusaRequest<{ csv: string }>, res: MedusaRespo
|
||||
}
|
||||
|
||||
const categoryByName = new Map(
|
||||
categories.filter((c) => c.name).map((c) => [c.name.toLowerCase().trim(), c.id]),
|
||||
categories.filter((c) => c.name).map((c) => [normalizeTitleForMatch(c.name), c.id]),
|
||||
);
|
||||
const collectionByName = new Map(
|
||||
collections.filter((c) => c.title).map((c) => [c.title.toLowerCase().trim(), c.id]),
|
||||
collections.filter((c) => c.title).map((c) => [normalizeTitleForMatch(c.title), c.id]),
|
||||
);
|
||||
const productByTitle = new Map(
|
||||
existingProducts.filter((p) => p.title).map((p) => [p.title.toLowerCase().trim(), p]),
|
||||
existingProducts.filter((p) => p.title).map((p) => [normalizeTitleForMatch(p.title), p]),
|
||||
);
|
||||
|
||||
const results: RowResult[] = [];
|
||||
@@ -228,7 +255,7 @@ export async function POST(req: MedusaRequest<{ csv: string }>, res: MedusaRespo
|
||||
},
|
||||
});
|
||||
|
||||
const existingProduct = productByTitle.get(title.toLowerCase());
|
||||
const existingProduct = productByTitle.get(normalizeTitleForMatch(title));
|
||||
if (existingProduct) {
|
||||
await updateProductsWorkflow(req.scope).run({
|
||||
input: {
|
||||
@@ -262,7 +289,7 @@ export async function POST(req: MedusaRequest<{ csv: string }>, res: MedusaRespo
|
||||
if (!inventoryItemId) throw new Error("No se pudo crear el registro de inventario");
|
||||
|
||||
const categoryName = row.categoria?.trim();
|
||||
let categoryId = categoryName ? categoryByName.get(categoryName.toLowerCase()) : undefined;
|
||||
let categoryId = categoryName ? categoryByName.get(normalizeTitleForMatch(categoryName)) : undefined;
|
||||
if (categoryName && !categoryId) {
|
||||
const { result: createdCategories } = await createProductCategoriesWorkflow(req.scope).run({
|
||||
input: {
|
||||
@@ -272,11 +299,11 @@ export async function POST(req: MedusaRequest<{ csv: string }>, res: MedusaRespo
|
||||
},
|
||||
});
|
||||
categoryId = createdCategories[0].id;
|
||||
categoryByName.set(categoryName.toLowerCase(), categoryId);
|
||||
categoryByName.set(normalizeTitleForMatch(categoryName), categoryId);
|
||||
}
|
||||
|
||||
const collectionName = row.coleccion?.trim();
|
||||
let collectionId = collectionName ? collectionByName.get(collectionName.toLowerCase()) : undefined;
|
||||
let collectionId = collectionName ? collectionByName.get(normalizeTitleForMatch(collectionName)) : undefined;
|
||||
if (collectionName && !collectionId) {
|
||||
const { result: createdCollections } = await createCollectionsWorkflow(req.scope).run({
|
||||
input: {
|
||||
@@ -286,7 +313,7 @@ export async function POST(req: MedusaRequest<{ csv: string }>, res: MedusaRespo
|
||||
},
|
||||
});
|
||||
collectionId = createdCollections[0].id;
|
||||
collectionByName.set(collectionName.toLowerCase(), collectionId);
|
||||
collectionByName.set(normalizeTitleForMatch(collectionName), collectionId);
|
||||
}
|
||||
|
||||
await createProductsWorkflow(req.scope).run({
|
||||
|
||||
@@ -81,4 +81,70 @@
|
||||
hacía falta agregar una clave nueva) — un script de setup inicial y
|
||||
uno de "agregar una clave a un secret ya vivo" deberían haber sido
|
||||
dos herramientas distintas desde el principio, no una sola que hay
|
||||
que saber cuándo NO correr completa.
|
||||
que saber cuándo NO correr completa.
|
||||
|
||||
## CSS Grid: un scroller horizontal sin `min-w-0` puede desbordar todo el grid
|
||||
|
||||
Encontrado en producción (2026-09-13/14) en la página de producto del
|
||||
storefront: un síntoma que parecía un bug de imágenes ("se ve como si
|
||||
abriera en tamaño real" en Chrome/Edge Android) resultó ser, en
|
||||
realidad, un desborde de layout. El carrusel horizontal de "También te
|
||||
puede gustar" (`overflow-x-auto`, sin `flex-wrap`) vivía como hijo
|
||||
directo de un CSS Grid. Por spec, el tamaño mínimo automático de un
|
||||
item de grid es el tamaño mínimo de su contenido **a menos que el
|
||||
propio item** tenga `overflow` distinto de `visible` — y como el `div`
|
||||
que envolvía al carrusel no lo tenía, el ancho mínimo intrínseco de
|
||||
*todas* las tarjetas del carrusel sin wrappear se contaba como el
|
||||
mínimo de esa columna, forzando **todo el grid compartido** (imagen,
|
||||
título, specs) a ~800px de ancho en un viewport real de ~380px.
|
||||
|
||||
**Lección reusable:** cualquier fila con scroll horizontal (carrusel,
|
||||
tira de miniaturas, chips de filtro) que sea hijo directo de un CSS
|
||||
Grid necesita `min-w-0` en su wrapper (o `overflow` seteado
|
||||
directamente en el item de grid) — si no, su contenido sin wrappear
|
||||
puede reventar el ancho de todo el grid en silencio, y el síntoma
|
||||
visible no tiene nada que ver con imágenes ni con DPR. Diagnosticado
|
||||
en vivo comparando `window.innerWidth` (reportaba el ancho
|
||||
desbordado) contra el viewport real del dispositivo.
|
||||
|
||||
## Rediseño de página de producto y correcciones de datos reales (2026-09-13/14)
|
||||
|
||||
Sesión de correcciones guiadas por capturas reales del usuario en
|
||||
navegadores reales (no reportes abstractos):
|
||||
|
||||
- **Bodega mal configurada:** `ENVIOCLICK_ORIGIN_DANE_CODE` apuntaba a
|
||||
Bogotá (`11001000`) cuando la bodega real está en Santiago de Cali,
|
||||
Valle del Cauca (`76001000`) — afectaba el cálculo real de envío
|
||||
(EnvíoClick) en cada checkout. Corregido en el ConfigMap del
|
||||
frontend, en `commerce-backend/medusa-config.ts` (incluyendo un
|
||||
fallback obsoleto que también apuntaba a Bogotá) y en el registro
|
||||
real de la bodega en Medusa (vía admin temporal, mismo patrón que el
|
||||
resto del lab para ediciones puntuales en vivo).
|
||||
- **Redes sociales y páginas de ayuda reales:** el footer enlazaba
|
||||
cuentas placeholder; se verificó en vivo la cuenta real de Instagram
|
||||
antes de linkearla (la primera sugerida no era la correcta) y se
|
||||
activaron los enlaces reales de Instagram/Facebook/WhatsApp, además
|
||||
de las páginas `/cambios` y `/preguntas-frecuentes` con contenido
|
||||
real (ya existían en código, no estaban enlazadas).
|
||||
- **Zoom estilo Amazon en la galería de producto:** iteración en tres
|
||||
pasos hasta llegar a lo pedido — `transform: scale()` sobre una
|
||||
imagen ya recortada por `object-cover` (no funcionaba: lo recortado
|
||||
nunca estaba en el DOM), superposición in-place (técnicamente
|
||||
correcta pero no la UX pedida), y finalmente una lupa que sigue el
|
||||
cursor más un panel lateral que magnifica esa región — el patrón
|
||||
real de Amazon. El tamaño/posición del panel se calcula en JS contra
|
||||
`window.innerWidth/innerHeight` en cada `mousemove` (no un
|
||||
breakpoint CSS fijo), con fallback a la superposición in-place si no
|
||||
hay espacio.
|
||||
- **Barra flotante de "Añadir al carrito" en mobile: removida, no
|
||||
reajustada.** Pasó por varias iteraciones de lógica de
|
||||
mostrar/ocultar (con `IntersectionObserver`) que seguían
|
||||
superponiéndose a otro contenido en capturas reales del usuario. En
|
||||
vez de seguir ajustando el umbral, se eliminó el componente por
|
||||
completo y se dejó el botón inline normal, visible también en
|
||||
mobile — el mismo patrón que usa la mayoría de tiendas. Verificado
|
||||
con Playwright: cero elementos `position: fixed` en la página de
|
||||
producto.
|
||||
- **Ambiente de Staging + versión real en el footer:** ver
|
||||
[ADR 0011](../decisiones/0011-ambiente-staging.md) para el detalle
|
||||
completo.
|
||||
@@ -230,9 +230,15 @@ falla:
|
||||
contenedor.
|
||||
- **Existencia de la sesión tmux `lab`** (`tmux has-session`): si no
|
||||
existe, **sí se recrea automáticamente** (`tmux new-session -d -s lab
|
||||
-c <WORKDIR>`) — es solo recuperar el propio entorno de trabajo del
|
||||
contenedor, no tocar infraestructura externa, así que es seguro
|
||||
hacerlo sin intervención humana.
|
||||
-c <WORKDIR>`, seguido de `tmux send-keys -t lab claude --remote-control
|
||||
--continue Enter` — ver gotcha "sesión recreada pero sin Claude Code
|
||||
adentro" más abajo, y los gotchas de Remote Control y `--continue`
|
||||
agregados el 2026-09-20) — es solo recuperar el propio entorno de
|
||||
trabajo del contenedor, no tocar infraestructura externa, así que es
|
||||
seguro hacerlo sin intervención humana. Si `WORKDIR` todavía no está
|
||||
montado (ver gotcha siguiente),
|
||||
no se crea la sesión esa vuelta — se reintenta en el próximo ciclo en
|
||||
vez de crearla mal posicionada.
|
||||
|
||||
Todo evento (chequeo ok, falla de red, sesión recreada) queda en
|
||||
`/DATA/AppData/k3d-lab-v3/tmux-host/logs/health.log` — persistente en
|
||||
@@ -247,14 +253,161 @@ contenedor con `-e HEALTHCHECK_INTERVAL_SEC=60` (solo para pruebas en un
|
||||
contenedor descartable — nunca cambiar esto en el `tmux-host` real sin
|
||||
volver a 1800 después).
|
||||
|
||||
## Gotcha: sesión recreada, pero sin Claude Code adentro (reinicio completo del NAS, 2026-09-19)
|
||||
|
||||
Distinto al caveat de persistencia de sesión de más abajo (ese cubre "el
|
||||
*proceso* muere pero el contenedor sigue vivo" — cerrar la sesión `lab` a
|
||||
mano, o un crash del propio `claude`). Este es otro escenario: **el HOST
|
||||
completo se apaga y vuelve a arrancar desde cero** (corte de luz, no un
|
||||
`docker restart` ordenado) — reordena cuándo arrancan Docker, la red y los
|
||||
montajes de `/DATA`/`/media/HDD-Storage` entre sí, algo que un
|
||||
`docker restart` normal nunca ejercita porque el host ya estaba arriba.
|
||||
|
||||
Tras un corte de luz real, se encontró que el usuario se reconectó a la
|
||||
sesión `lab` y encontró un shell vacío, no Claude Code corriendo. Dos
|
||||
causas distintas, ambas corregidas:
|
||||
|
||||
1. **La sesión tmux SÍ se recreaba sola** (confirmado en `health.log`: el
|
||||
contenedor arrancó y la sesión `lab` quedó recreada en cuestión de
|
||||
segundos), pero `check_tmux_session()` nunca lanzó `claude` adentro —
|
||||
solo garantizaba que existiera el *shell*, no que el proceso de Claude
|
||||
Code estuviera corriendo. El diagrama de arquitectura de este mismo doc
|
||||
siempre mostró `T --> CC["proceso claude"]` como si fuera automático,
|
||||
pero el código nunca lo implementó — el hueco quedó invisible mientras
|
||||
nadie pasó por un reinicio completo real del NAS (en un `docker
|
||||
restart` normal, el `claude` que ya estaba corriendo dentro de la
|
||||
sesión simplemente sigue vivo, porque ahí el proceso *no* muere).
|
||||
**Fix:** `check_tmux_session()` ahora llama `launch_claude_in_session()`
|
||||
(`tmux send-keys -t lab claude Enter`) justo después de crear la sesión
|
||||
— solo cuando la crea de cero, nunca sobre una sesión que ya existe (no
|
||||
se reinyecta `claude` sobre un pane con trabajo en curso).
|
||||
2. **Riesgo de carrera no ejercitado todavía, pero real:** `WORKDIR`
|
||||
(`/DATA/AppData/k3d-lab-v3/lab-cruzcloud`) llega vía el symlink absoluto
|
||||
`/DATA/AppData -> /media/HDD-Storage/AppData` (ver el gotcha del symlink
|
||||
más arriba). Si el storage externo (`/media/HDD-Storage`) tarda en
|
||||
montarse en el host más de lo que tarda Docker en arrancar este
|
||||
contenedor — plausible después de un corte de luz, no de un simple
|
||||
restart —, `mkdir -p "$LOG_DIR"` fallaba en silencio (`set -u` sin
|
||||
`-e`) y ese arranque quedaba *sin ningún rastro* en el log, y
|
||||
`tmux new-session -c "$WORKDIR"` habría creado la sesión mal
|
||||
posicionada (o fallado) contra un directorio que todavía no existe.
|
||||
**Fix:** `wait_for_workdir()` reintenta con backoff (pasos de 5s, hasta
|
||||
`STARTUP_MOUNT_WAIT_MAX_SEC=120` por defecto) a que `WORKDIR` exista
|
||||
antes de tocar el log o crear la sesión; si sigue sin existir pasado
|
||||
ese máximo, se registra el error explícitamente y se reintenta en el
|
||||
siguiente ciclo (30 min) en vez de fallar una sola vez y quedarse así.
|
||||
|
||||
**Validado en vivo:** imagen reconstruida (`tmux-host:test-fix`) y probada
|
||||
en un contenedor descartable aparte (mismos mounts de `/DATA` y
|
||||
`/media/HDD-Storage`, sin tocar el `tmux-host` real) — `docker stop` +
|
||||
`docker start` (arranque desde cero, no `restart`) recreó la sesión `lab`
|
||||
con `claude` corriendo adentro sin intervención manual, en ~1 segundo.
|
||||
|
||||
## Gotcha: Remote Control nunca se activaba solo (2026-09-20)
|
||||
|
||||
Con la sesión `lab` recreándose sola y `claude` corriendo adentro (gotcha
|
||||
anterior), quedaba un problema distinto y más sutil: la sesión servía
|
||||
perfecto para trabajar vía `docker exec`/`lab-session.sh`, pero nunca
|
||||
aparecía como controlable remotamente en claude.ai — como si Remote
|
||||
Control jamás se hubiera activado.
|
||||
|
||||
**Causa raíz (confirmado en vivo 2026-09-20):** Remote Control es
|
||||
opt-in por lanzamiento, vía el flag `claude --remote-control [nombre]`
|
||||
de la CLI. Nada en `launch_claude_in_session()` lo pasaba nunca, así
|
||||
que `remoteEnabled` en `~/.claude.json` quedaba en `false` para siempre
|
||||
porque nadie lo pedía. No existe un paso separado de confirmación
|
||||
"¿Activar Remote Control? y/n" más allá del diálogo normal de
|
||||
confianza del folder (una hipótesis inicial de que sí existía tal paso,
|
||||
y de que había una clave `remoteControlAtStartup` en `settings.json`,
|
||||
resultó falsa al revisar directamente los archivos de configuración
|
||||
reales).
|
||||
|
||||
**Cómo se detectó:** lanzar `claude --remote-control` a mano (en una
|
||||
ventana de tmux en foreground, no vía un `!` de un solo uso que se
|
||||
mata al terminar) sí registró la sesión y apareció en la lista de
|
||||
sesiones activas de claude.ai como `zimaos-quizzical` — el hostname del
|
||||
contenedor (`ZimaOS`) se usa como prefijo por defecto del nombre de
|
||||
sesión remota.
|
||||
|
||||
**Fix:** cambiar la línea `tmux send-keys` en
|
||||
`launch_claude_in_session()` para lanzar `claude --remote-control` en
|
||||
vez de `claude` a secas.
|
||||
|
||||
**Validado en vivo:** tras reconstruir la imagen y swapear el
|
||||
contenedor real (ver el gotcha del swap más abajo), `health.log`
|
||||
registró `'claude --remote-control' lanzado...` al recrear la sesión, y
|
||||
el pane de tmux mostró `/remote-control is active` con un link de
|
||||
sesión válido — sin que nadie lo activara a mano.
|
||||
|
||||
## Gotcha: cada recreación de la sesión empezaba una conversación en blanco (2026-09-20)
|
||||
|
||||
Con Remote Control ya activándose solo, quedaba una fricción menor pero
|
||||
molesta: cada vez que la sesión `lab` se recreaba (reinicio del NAS,
|
||||
`docker rm` + recrear, o el propio healthcheck reaccionando a un
|
||||
crash), `launch_claude_in_session()` arrancaba una conversación
|
||||
completamente nueva. El contexto de trabajo en curso —no las
|
||||
credenciales ni la confianza del folder, que sí persisten vía volumen—
|
||||
se perdía cada vez, sin relación con los límites de cuota (ver la nota
|
||||
al principio de este doc).
|
||||
|
||||
**Fix:** agregar `-c`/`--continue` (retoma la conversación más reciente
|
||||
en el directorio de trabajo actual) al comando de lanzamiento:
|
||||
`claude --remote-control --continue`. Si no hay ninguna conversación
|
||||
previa en `WORKDIR`, el propio CLI arranca una nueva sin error — no
|
||||
hace falta lógica condicional en el script.
|
||||
|
||||
**Validado en vivo — la prueba más fuerte posible:** el swap que
|
||||
desplegó este mismo cambio (ver gotcha siguiente) mató la sesión de
|
||||
Claude Code que lo estaba ejecutando. Del otro lado, la *misma
|
||||
conversación* siguió sin cortarse, con el mismo ID de sesión, sin
|
||||
ninguna intervención manual — evidencia directa de que `--continue`
|
||||
efectivamente retoma la conversación y no solo que el contenedor volvió
|
||||
a levantar.
|
||||
|
||||
## Gotcha: actualizar la imagen de `tmux-host` en caliente mata al propio proceso que hace el swap (2026-09-20)
|
||||
|
||||
Este contenedor tiene un problema estructural para actualizarse a sí
|
||||
mismo: el proceso de Claude Code que ejecutaría `docker stop tmux-host
|
||||
&& docker rm tmux-host && docker run ... tmux-host:nueva-imagen` corre
|
||||
*dentro* del propio contenedor `tmux-host` — el primer `docker stop`
|
||||
mata su propia shell a mitad de la secuencia, dejando el swap a medio
|
||||
terminar (visto en vivo: un primer intento así falló en silencio).
|
||||
|
||||
**Fix:** correr la secuencia de swap desde un contenedor *sibling*
|
||||
efímero, no desde dentro de `tmux-host`:
|
||||
|
||||
```bash
|
||||
docker run --rm \
|
||||
-v /var/run/docker.sock:/var/run/docker.sock \
|
||||
-v /DATA/AppData/k3d-lab-v3/tmux-host:/tmux-host-cfg \
|
||||
docker:cli sh /tmux-host-cfg/swap-retry.sh
|
||||
```
|
||||
|
||||
Como este helper solo comparte el socket de Docker (no vive *dentro*
|
||||
del contenedor `tmux-host`), detener y recrear `tmux-host` no lo afecta
|
||||
a él ni al proceso que lo lanzó desde el host. El script
|
||||
(`swap-retry.sh`, junto a sus logs, vive en
|
||||
`/DATA/AppData/k3d-lab-v3/tmux-host/` — mismo path físico que
|
||||
`/media/HDD-Storage/AppData/k3d-lab-v3/tmux-host/`, ver el gotcha del
|
||||
symlink más arriba) deja un log fechado por corrida en `logs/` con el
|
||||
`docker ps` y un `grep` sobre `healthcheck.sh` dentro de la imagen
|
||||
nueva, para verificar sin tener que adivinar si el fix realmente quedó
|
||||
adentro.
|
||||
|
||||
**Validado en vivo dos veces (2026-09-20):** el fix de Remote Control y
|
||||
el de `--continue` se desplegaron cada uno con este mismo mecanismo,
|
||||
ambos confirmados por `docker ps` (imagen e `CreatedAt` nuevos) y por
|
||||
`health.log` mostrando la línea de lanzamiento esperada inmediatamente
|
||||
después del swap.
|
||||
|
||||
## Qué sobrevive a qué
|
||||
|
||||
| Evento | ¿Sobrevive la sesión tmux (`lab`)? | ¿Sobrevive el login de Claude Code? |
|
||||
|---|---|---|
|
||||
| Corte de SSH / cerrar la terminal / cambiar de dispositivo | Sí — es exactamente el caso que este setup resuelve | Sí |
|
||||
| `docker restart tmux-host` (reinicio del NAS) | **No al toque, pero se auto-recupera solo** — el proceso `tmux server` muere con el contenedor; `healthcheck.sh` la recrea en el próximo ciclo (máx. 30 min) sin intervención, o `lab-session.sh` la crea al instante si alguien se reconecta antes | Sí, vía el volumen montado en `/root/.claude` |
|
||||
| `docker rm` + recrear el contenedor desde cero | No, pero igual se auto-recupera sola (mismo mecanismo de arriba) | Sí, vía el mismo volumen — no hace falta volver a autenticar ni reconfirmar el diálogo de confianza del folder (ver el mount de `claude-code-home.json`) |
|
||||
| Falla de red real del host/NAS | N/A | N/A — `healthcheck.sh` la detecta y notifica, pero no la corrige (fuera de su alcance) |
|
||||
| Evento | ¿Sobrevive la sesión tmux (`lab`)? | ¿Sobrevive el login de Claude Code? | ¿Retoma la misma conversación? |
|
||||
|---|---|---|---|
|
||||
| Corte de SSH / cerrar la terminal / cambiar de dispositivo | Sí — es exactamente el caso que este setup resuelve | Sí | Sí — el proceso `claude` ni se entera |
|
||||
| `docker restart tmux-host` (reinicio del NAS) | **No al toque, pero se auto-recupera solo** — el proceso `tmux server` muere con el contenedor; `healthcheck.sh` la recrea en el próximo ciclo (máx. 30 min) sin intervención, o `lab-session.sh` la crea al instante si alguien se reconecta antes | Sí, vía el volumen montado en `/root/.claude` | Sí, desde el 2026-09-20 — `--continue` retoma la conversación más reciente en `WORKDIR` (ver gotcha arriba) |
|
||||
| `docker rm` + recrear el contenedor desde cero | No, pero igual se auto-recupera sola (mismo mecanismo de arriba) | Sí, vía el mismo volumen — no hace falta volver a autenticar ni reconfirmar el diálogo de confianza del folder (ver el mount de `claude-code-home.json`) | Sí, mismo mecanismo — validado en vivo durante el swap de imagen del 2026-09-20 |
|
||||
| Falla de red real del host/NAS | N/A | N/A — `healthcheck.sh` la detecta y notifica, pero no la corrige (fuera de su alcance) | N/A |
|
||||
|
||||
## Validación en vivo
|
||||
|
||||
|
||||
@@ -328,7 +328,69 @@ compartido) para que el contenedor vea los archivos de verdad.
|
||||
`<select>` del checkout. Ningún banco PSE real tiene código "0" —
|
||||
se filtra explícitamente en `pse-banks/route.ts`.
|
||||
|
||||
### Pendiente — checklist ampliado para la próxima sesión con navegador
|
||||
Ampliado 2026-09-10 con el resto de la superficie real del storefront
|
||||
(4 páginas + `components/{account,cart,catalog,checkout,home,layout,
|
||||
product,search,wishlist}`), agrupado por riesgo, y ejecutado el mismo
|
||||
día contra `shop.cruzcloud.net` con el harness de `.browser-scratch/`.
|
||||
|
||||
### RESUELTO (2026-09-11) — segunda tanda con navegador real
|
||||
|
||||
Resultados reales, uno por uno (código de cada prueba en
|
||||
`checks.mjs`):
|
||||
|
||||
- **`/checkout` con carrito vacío (ítem 9) — OK.** Muestra "Tu carrito
|
||||
está vacío" con link a `/catalog`, no rompe.
|
||||
- **Fallback "Confirmar por WhatsApp" (ítem 10) — OK.** Abre
|
||||
`api.whatsapp.com/send?phone=...` en pestaña nueva con el mensaje del
|
||||
pedido armado correctamente.
|
||||
- **Filtros de `/catalog` (ítem 11) — OK.** Un filtro de categoría real
|
||||
("Figuras de acción") redujo los productos mostrados de 18 a 4. Nota:
|
||||
estos filtros (categoría/marca/edad dentro de la barra lateral) son
|
||||
**estado de cliente, no reflejado en la URL** — distinto de los
|
||||
filtros por query param (`?category=...`) que sí usa la navegación
|
||||
del header. Ninguno de los dos es un bug, son dos mecanismos
|
||||
distintos coexistiendo; queda como nota para no confundirlos.
|
||||
- **Búsqueda sin resultados (ítem 12) — BUG real encontrado y
|
||||
corregido.** El dropdown de autocompletado no mostraba absolutamente
|
||||
nada (ni "Buscando…" ni ningún mensaje) cuando una búsqueda de 2+
|
||||
caracteres no tenía coincidencias — el usuario se quedaba sin
|
||||
feedback de que su búsqueda había corrido. Corregido en
|
||||
`predictive-search.tsx`: ahora muestra "No encontramos resultados
|
||||
para…" con el botón de buscar en todo el catálogo.
|
||||
- **Slug de producto inválido (ítem 14) — hallazgo, no corregido
|
||||
todavía.** La página sí muestra el contenido correcto de "Este tesoro
|
||||
no existe" (el código ya llama a `notFound()` de
|
||||
`next/navigation`, como corresponde), **pero la respuesta HTTP real
|
||||
es `200`, no `404`** — confirmado independientemente con `curl`, no
|
||||
es un artefacto de Playwright. No se investigó a fondo en qué capa se
|
||||
pierde el status real (Next.js standalone vs. alguna capa de
|
||||
Cloudflare/NPM/Traefik en el medio, ver
|
||||
`incidente-cloudflared-504-quic-2026-08.md` para precedente de
|
||||
problemas de red en esa cadena) — bajo impacto en la experiencia del
|
||||
usuario (el contenido es correcto), pero sí afecta SEO/crawlers.
|
||||
Queda como pendiente de investigar, no de corregir a ciegas.
|
||||
- **Zoom interactivo del producto (ítem 15) — OK.** Hover-zoom y el
|
||||
modal de click-to-zoom siguen funcionando igual tras los cambios
|
||||
recientes al storefront.
|
||||
- **Wishlist/favoritos (ítem 17) — OK.** Agregar a favoritos +
|
||||
recargar la página: persiste correctamente vía `localStorage`.
|
||||
- **Otros viewports, tablet y desktop grande (ítem 19) — OK.**
|
||||
Capturas guardadas, sin problemas de layout visibles en el banner ni
|
||||
el header en esos anchos.
|
||||
- **Página 404 genérica (ítem 20) — OK.** Una ruta que no existe cae en
|
||||
`not-found.tsx` con status HTTP `404` real (a diferencia del caso de
|
||||
producto inválido arriba — la diferencia de comportamiento entre
|
||||
ambos casos es justamente lo que quedó como pendiente de investigar).
|
||||
- **Accesibilidad básica (ítem 21) — OK, parcial.** El buscador es
|
||||
alcanzable navegando solo con `Tab` desde el body, sin mouse. Falta
|
||||
probar el resto del flujo de checkout por teclado.
|
||||
- **Registro/login con email y contraseña (parte del ítem 16) — OK.**
|
||||
Crear cuenta con nombre/apellido/email/contraseña funciona y deja la
|
||||
sesión iniciada. La parte de Google OAuth sigue sin probarse (necesita
|
||||
una cuenta de Google real, no algo que se pueda automatizar sin
|
||||
credenciales).
|
||||
|
||||
### Pendiente — lo que queda sin probar
|
||||
|
||||
3. **Resiliencia del fallback de envío:** forzar una dirección o
|
||||
producto que haga fallar la cotización real de EnvíoClick (ej. un
|
||||
@@ -340,17 +402,72 @@ compartido) para que el contenedor vea los archivos de verdad.
|
||||
conseguir credenciales de prueba reales de PSE (persona/clave de
|
||||
algún banco del ambiente `desarrollo`), completar el login bancario
|
||||
de punta a punta, y confirmar que el webhook mueve la orden a
|
||||
completada sin intervención manual — hoy solo se confirmó que el
|
||||
endpoint responde `200` a un `POST` sintético y que la redirección
|
||||
al banco ocurre, no el ciclo completo con notificación real.
|
||||
7. **Login/registro de cliente real** (Google OAuth y con contraseña) —
|
||||
nunca ejercitado con un navegador real en ninguna sesión.
|
||||
8. **Bug conocido de Firefox Android** (ver memoria de proyecto,
|
||||
`known-bug-firefox-android-layout`): el layout se corta/desplaza
|
||||
solo en Firefox Android — Playwright puede emular Android/Chrome
|
||||
pero no Firefox; si se necesita reproducir esto específicamente hace
|
||||
falta `playwright-firefox` o un dispositivo real.
|
||||
9. **Otros viewports** (tablet, desktop grande) para el banner y el
|
||||
buscador — solo se probaron mobile (iPhone 13) y desktop 1280px.
|
||||
10. **Flujo de wishlist/favoritos y el diálogo de cuenta** — no
|
||||
ejercitados todavía con navegador real.
|
||||
completada sin intervención manual.
|
||||
7. **Pago cancelado o rechazado en el banco PSE:** confirmar que la
|
||||
orden no queda en un estado ambiguo ("colgada" en
|
||||
`pending_authorization` para siempre).
|
||||
8. **Persona jurídica + NIT** en el formulario de PSE — solo se probó
|
||||
persona natural con cédula.
|
||||
13. **Selección de variantes** (cuando un producto tiene más de una) y
|
||||
que el precio/disponibilidad mostrado cambie según la variante
|
||||
elegida — ningún producto probado hasta ahora tenía más de una
|
||||
variante real.
|
||||
14b. **Investigar por qué `/product/<slug-inválido>` responde HTTP 200**
|
||||
en vez de 404 pese a llamar `notFound()` correctamente (ver
|
||||
hallazgo arriba) — identificar la capa responsable antes de tocar
|
||||
código.
|
||||
16b. **Login con Google (OAuth completo)** incluyendo
|
||||
`/auth/google/callback` — necesita una cuenta de Google real de
|
||||
prueba.
|
||||
18. **Bug conocido de Firefox Android** (ver memoria de proyecto,
|
||||
`known-bug-firefox-android-layout`) — Playwright puede emular
|
||||
Android/Chrome pero no Firefox; hace falta `playwright-firefox` o
|
||||
un dispositivo real.
|
||||
21b. **Accesibilidad del checkout completo** por teclado (no solo el
|
||||
buscador) — tab order y foco visible en los 3 pasos.
|
||||
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
|
||||
arreglada en código (2026-09-20), pendiente confirmar con una
|
||||
edición real.** `medusaFetch()` en `lib/headless/providers/medusa.ts`
|
||||
usaba `next:{revalidate:60,tags:["medusa-products"]}`, caché en
|
||||
memoria por-pod que sobrevivía a `force-dynamic`. Con
|
||||
`frontend-deploy` en 2 réplicas sin cacheHandler compartido, el POST
|
||||
de `/api/revalidate` del subscriber de Medusa solo llegaba a uno de
|
||||
los dos pods — el otro seguía sirviendo datos viejos hasta que el
|
||||
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
|
||||
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
|
||||
producción. Falta la prueba con una edición manual real (vía
|
||||
`temp-admin`/`ari-shopping-tienda` en memoria) para confirmar que
|
||||
el síntoma original desapareció, no solo que la causa técnica tiene
|
||||
sentido.
|
||||
23. **En mobile (viewport real de 320px), la página se cortaba y
|
||||
obligaba a scrollear horizontalmente — RESUELTO 2026-09-20.**
|
||||
Reportado por el usuario con captura real de `shop.cruzcloud.net`.
|
||||
**No era la galería de imágenes** (esa clase de bug con `min-w-0`
|
||||
ya estaba corregida desde la sesión anterior, confirmado en vivo
|
||||
que seguía sana). Dos causas reales encontradas por separado,
|
||||
reproducidas con Playwright contra un viewport de 320×480 real:
|
||||
- **Footer, bloque "Newsletter"** (`components/layout/footer.tsx`):
|
||||
el párrafo de texto, sin `min-w-0`, le pedía al grid item su
|
||||
ancho de una sola línea sin wrappear (~335px, medido en vivo)
|
||||
en vez de dejarse achicar — forzaba la única columna del grid
|
||||
(`sm:grid-cols-2 lg:grid-cols-4`, sin `grid-cols-1` propio pero
|
||||
con auto-stacking correcto por debajo de `sm`) a 335px de ancho,
|
||||
arrastrando la página entera. Mismo bug de fondo que el ya
|
||||
documentado en la galería, en un grid en vez de un
|
||||
`overflow-x-auto`.
|
||||
- **Navbar, fila de "Departamentos"/"Todo"** (`components/layout/navbar.tsx`):
|
||||
los 3 items de esa fila (botón de menú, "Departamentos", "Todo")
|
||||
tenían los tres `shrink-0` — ninguno podía achicarse, así que si
|
||||
los tres no entraban en 320px, desbordaban sí o sí (`min-w-0` no
|
||||
aplica cuando el ítem no está pensado para achicarse). Fix:
|
||||
ocultar el texto "Departamentos" en mobile (queda ícono + flecha,
|
||||
con `aria-label` agregado para no perder accesibilidad), libera
|
||||
espacio suficiente para que quepan los tres.
|
||||
Verificado en vivo con Playwright (viewport 320×480 real, no
|
||||
emulación de dispositivo con DPR) contra staging y producción
|
||||
después del deploy: `document.documentElement.scrollWidth ===
|
||||
clientWidth` en home y en página de producto, en ambos ambientes —
|
||||
sin overflow.
|
||||
@@ -0,0 +1,115 @@
|
||||
# ADR 0011 — Ambiente de Staging para el storefront
|
||||
|
||||
**Estado:** aceptada — desplegado y verificado en vivo
|
||||
**Fecha:** 2026-09-14
|
||||
|
||||
## Contexto
|
||||
|
||||
Hasta este punto, todo cambio de frontend se probaba directamente
|
||||
contra `shop.cruzcloud.net` (producción): el mismo push a `main` que
|
||||
disparaba el build de imagen también actualizaba el manifiesto real
|
||||
que sirve tráfico. Cada iteración de UI (rediseño de la página de
|
||||
producto, zoom estilo Amazon, ajustes mobile) implicaba validar en
|
||||
vivo sobre el sitio real, sin una red de contención — un cambio roto
|
||||
llegaba directo al usuario final antes de poder revertirlo.
|
||||
|
||||
El usuario pidió explícitamente un ambiente de Staging para dejar de
|
||||
probar en producción ("Me interesa crear el ambiente Staging para no
|
||||
estar probando en Prod").
|
||||
|
||||
## Opciones consideradas
|
||||
|
||||
| Opción | A favor | En contra |
|
||||
|---|---|---|
|
||||
| **Namespace + Ingress propios, backend compartido con Prod** — elegida | Cero costo adicional de Postgres/Medusa/Meilisearch en un NAS ya ajustado de recursos; rápido de armar reusando el mismo patrón workload+governance de `ecommerce` | Staging y Prod comparten datos reales (catálogo, inventario, órdenes) — no aísla pruebas que muten datos de negocio |
|
||||
| Backend propio para Staging (Postgres/Medusa/Meilisearch duplicados) | Aislamiento completo, se podrían probar migraciones de datos sin riesgo | Duplica el consumo de recursos de la NAS sin necesidad todavía (decisión explícita del usuario de descartar esto por ahora) |
|
||||
| Feature flags / preview deploys por PR en vez de un ambiente fijo | Sin infraestructura nueva que mantener | Gitea Actions en este lab no tiene un mecanismo de preview-per-branch ya armado; más esfuerzo de tooling que un namespace nuevo |
|
||||
|
||||
## Decisión
|
||||
|
||||
Namespace `ecommerce-staging` nuevo, con su propio Ingress
|
||||
(`staging.cruzcloud.net`) y su propio Deployment de frontend (misma
|
||||
imagen Docker que Prod), pero apuntando al **mismo backend Medusa** que
|
||||
producción (`medusa-svc.ecommerce.svc.cluster.local:9000` — FQDN
|
||||
cross-namespace, un nombre corto solo resuelve dentro del mismo
|
||||
namespace). Flujo de promoción: **todo push a `main` despliega a
|
||||
Staging automáticamente**; promover a Producción es una acción manual
|
||||
separada (`scripts/promote-to-production.sh`).
|
||||
|
||||
## Qué se armó
|
||||
|
||||
- `apps-registry/workloads/ecommerce-staging/` — overlay Kustomize:
|
||||
1 réplica (no 2 como Prod — ambiente de verificación, no de tráfico
|
||||
real), requests/limits más chicos, mismo `securityContext` que Prod
|
||||
(mismas restricciones del Dockerfile: `runAsUser` numérico,
|
||||
`NET_BIND_SERVICE`, `readOnlyRootFilesystem`).
|
||||
- `apps-registry/apps/ecommerce-staging-app.yaml` +
|
||||
`ecommerce-staging-governance-app.yaml` — Applications de Argo CD
|
||||
(workload + governance separadas, misma convención que el resto del
|
||||
lab).
|
||||
- `platform-infra/ecommerce-staging-governance/` — ResourceQuota,
|
||||
LimitRange y NetworkPolicy propios del namespace nuevo.
|
||||
- `.gitea/workflows/build.yaml` — el paso de CD (`Actualizar
|
||||
Manifiesto GitOps`) ahora apunta a
|
||||
`workloads/ecommerce-staging/frontend/patches/deployment.yaml` en
|
||||
vez del manifiesto de Prod: cada push a `main` bumpea el tag de
|
||||
imagen (y, desde el mismo día, la variable `APP_VERSION` — ver nota
|
||||
de implementación) de Staging automáticamente.
|
||||
- `scripts/promote-to-production.sh` — lee el tag actualmente
|
||||
desplegado en Staging, pide confirmación explícita, y lo aplica al
|
||||
manifiesto real de Producción (`workloads/ecommerce/frontend/patches/deployment.yaml`).
|
||||
Nunca corre automáticamente: es la única vía para que un cambio
|
||||
llegue a `shop.cruzcloud.net`.
|
||||
- Secrets `commerce-storefront` y `gitea-registry-secret` copiados
|
||||
(no regenerados) desde el namespace `ecommerce` hacia
|
||||
`ecommerce-staging`, para que el frontend de Staging tenga las
|
||||
mismas credenciales reales contra el backend compartido.
|
||||
|
||||
## Descubrimiento de red real (no solo Kubernetes)
|
||||
|
||||
Confirmado en vivo el mismo día: aunque Cloudflare tiene un wildcard
|
||||
`*.cruzcloud.net`, eso **no alcanza** para exponer un Ingress nuevo. La
|
||||
cadena real es Cloudflare → túnel `cloudflared` → **Nginx Proxy
|
||||
Manager** → `k3d-lab-cluster-serverlb` (puerto 90) → Traefik → Service.
|
||||
Cada subdominio que ya funciona (`shop`, `commerce`, `media`, etc.)
|
||||
tiene su propia fila de Proxy Host en NPM apuntando a
|
||||
`192.168.68.61:90`. Un Ingress nuevo sin esa fila devuelve la página
|
||||
por defecto de NPM aunque Traefik ya esté enrutando bien internamente
|
||||
— verificado con `curl -H "Host: staging.cruzcloud.net"
|
||||
http://192.168.68.61:90/` (funcionaba) contra el dominio público
|
||||
(seguía devolviendo la página por defecto hasta que se agregó la fila
|
||||
en NPM). Ver [Red y exposición](../arquitectura/red-y-exposicion.md).
|
||||
Esta fila de NPM la agregó el usuario manualmente — no hay credenciales
|
||||
de ese panel disponibles para automatizarlo.
|
||||
|
||||
## Nota de implementación (2026-09-14): versión real visible en el footer
|
||||
|
||||
El footer mostraba un string fijo (`v4.1`), inútil para distinguir a
|
||||
simple vista si Staging y Producción están en la misma versión.
|
||||
Se agregó una variable de entorno `APP_VERSION` (leída en
|
||||
`footer.tsx` vía `process.env.APP_VERSION`, sin necesidad de prefijo
|
||||
`NEXT_PUBLIC_` por ser un Server Component) a ambos manifiestos de
|
||||
Deployment; el mismo paso del CD que bumpea el tag de imagen ahora
|
||||
también bumpea `APP_VERSION` (vía `perl -0777` para una sustitución
|
||||
multilínea, ya que el nombre y el valor de la variable viven en líneas
|
||||
distintas del YAML — un `sed` de una sola línea no alcanza).
|
||||
`promote-to-production.sh` hace el mismo bump al promover. Verificado
|
||||
en vivo con Playwright: tras el primer push, Staging mostró `v1.0.263`
|
||||
mientras Producción seguía en `v4.1` (la imagen vieja no leía la
|
||||
variable); tras promover, ambos ambientes quedaron visualmente
|
||||
distinguibles por versión real en su footer.
|
||||
|
||||
## Consecuencias
|
||||
|
||||
- Ganado: ningún cambio de frontend vuelve a validarse por primera vez
|
||||
contra tráfico real; Staging y Producción ahora se distinguen a
|
||||
simple vista por la versión mostrada en el footer.
|
||||
- Deuda aceptada: Staging comparte datos reales de negocio con
|
||||
Producción (mismo Medusa/Postgres) — una prueba que cree o modifique
|
||||
órdenes/inventario en Staging afecta datos reales. Si esto se vuelve
|
||||
un problema, la opción descartada de backend propio para Staging
|
||||
queda como camino de escalamiento.
|
||||
- Para revertir: apuntar de nuevo `MANIFEST_FILE`/`KUSTOMIZE_DIR` en
|
||||
`build.yaml` hacia `workloads/ecommerce/...` y eliminar el namespace
|
||||
`ecommerce-staging` — el namespace de Producción no cambia de forma
|
||||
en ningún momento de este ADR.
|
||||
@@ -97,6 +97,8 @@ nav:
|
||||
- "0007 · Identidad visual propia": decisiones/0007-identidad-visual-propia.md
|
||||
- "0008 · Identidad Navy & Coral": decisiones/0008-identidad-navy-coral.md
|
||||
- "0009 · Checkout real": decisiones/0009-checkout-real.md
|
||||
- "0010 · Pago y envío reales": decisiones/0010-pago-envio-reales-y-storefront.md
|
||||
- "0011 · Ambiente de Staging": decisiones/0011-ambiente-staging.md
|
||||
- Plantilla ADR: decisiones/plantilla-adr.md
|
||||
- Playbooks:
|
||||
- Crashloop Medusa: playbooks/incidente-crashloop-medusa.md
|
||||
|
||||
@@ -25,7 +25,7 @@ spec:
|
||||
runAsNonRoot: false
|
||||
containers:
|
||||
- name: app
|
||||
image: gitea.cruzcloud.net/devops/docs-portal:v1.0.237
|
||||
image: gitea.cruzcloud.net/devops/docs-portal:v1.0.269
|
||||
# El base saca TODAS las capabilities. Probado en vivo y
|
||||
# revertido: con drop: ALL (aunque se agregue NET_BIND_SERVICE
|
||||
# de vuelta) el propio entrypoint de nginx:1.27-alpine falla al
|
||||
|
||||
@@ -0,0 +1,51 @@
|
||||
apiVersion: kustomize.config.k8s.io/v1beta1
|
||||
kind: Kustomization
|
||||
|
||||
namespace: ecommerce-staging
|
||||
|
||||
resources:
|
||||
- https://gitea.cruzcloud.net/devops/platform-templates.git//k8s-manifests/base?ref=main
|
||||
|
||||
patches:
|
||||
# Mismo criterio que workloads/ecommerce/frontend/kustomization.yaml:
|
||||
# sin anchors/aliases de YAML a propósito (el parser de Gitea Actions no
|
||||
# los resuelve en "on:", pero acá aplica igual por consistencia con el
|
||||
# resto del repo). Ingress propio via ingress-staging.yaml en vez del
|
||||
# generico de 1-backend del base.
|
||||
- target:
|
||||
kind: Ingress
|
||||
name: app-ingress
|
||||
patch: |-
|
||||
$patch: delete
|
||||
apiVersion: networking.k8s.io/v1
|
||||
kind: Ingress
|
||||
metadata:
|
||||
name: app-ingress
|
||||
- target:
|
||||
kind: Deployment
|
||||
name: app-deployment
|
||||
path: patches/deployment.yaml
|
||||
- target:
|
||||
kind: Service
|
||||
name: app-svc
|
||||
path: patches/service.yaml
|
||||
|
||||
- target:
|
||||
kind: Deployment
|
||||
name: app-deployment
|
||||
patch: |-
|
||||
- op: replace
|
||||
path: /metadata/name
|
||||
value: frontend-deploy
|
||||
- op: replace
|
||||
path: /spec/template/spec/containers/0/ports
|
||||
value:
|
||||
- name: http
|
||||
containerPort: 80
|
||||
- target:
|
||||
kind: Service
|
||||
name: app-svc
|
||||
patch: |-
|
||||
- op: replace
|
||||
path: /metadata/name
|
||||
value: frontend-svc
|
||||
@@ -0,0 +1,89 @@
|
||||
apiVersion: apps/v1
|
||||
kind: Deployment
|
||||
metadata:
|
||||
name: app-deployment
|
||||
labels:
|
||||
app: frontend
|
||||
spec:
|
||||
# 1 réplica (no 2 como Producción): Staging es para verificar cambios
|
||||
# antes de promover, no para servir tráfico real -- alcanza, y deja más
|
||||
# margen de recursos en la NAS para el resto del cluster.
|
||||
replicas: 1
|
||||
strategy:
|
||||
type: RollingUpdate
|
||||
rollingUpdate:
|
||||
maxUnavailable: 0
|
||||
maxSurge: 1
|
||||
selector:
|
||||
matchLabels:
|
||||
app: frontend
|
||||
template:
|
||||
metadata:
|
||||
labels:
|
||||
app: frontend
|
||||
spec:
|
||||
securityContext:
|
||||
runAsUser: 1001
|
||||
containers:
|
||||
- name: app
|
||||
image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.273
|
||||
env:
|
||||
# Ver comentario en workloads/ecommerce/frontend/patches/deployment.yaml
|
||||
- name: APP_VERSION
|
||||
value: v1.0.273
|
||||
- name: HEADLESS_PROVIDER
|
||||
value: medusa
|
||||
# Backend COMPARTIDO con Producción a propósito (decisión del
|
||||
# usuario 2026-09-14: armar un backend propio para Staging
|
||||
# -Postgres/Medusa/Meilisearch- duplicaría el consumo de la
|
||||
# NAS sin necesidad todavía). FQDN cross-namespace: "ecommerce"
|
||||
# es el namespace real de medusa-svc -- el nombre corto
|
||||
# "medusa-svc" solo resuelve dentro del mismo namespace.
|
||||
- name: MEDUSA_BACKEND_URL
|
||||
value: http://medusa-svc.ecommerce.svc.cluster.local:9000
|
||||
- name: MEDUSA_REGION_ID
|
||||
value: reg_01KZT86WPAH7A7ZZRDSX3350V2
|
||||
envFrom:
|
||||
- secretRef:
|
||||
name: commerce-storefront
|
||||
readinessProbe:
|
||||
httpGet:
|
||||
path: /api/health
|
||||
port: http
|
||||
livenessProbe:
|
||||
httpGet:
|
||||
path: /api/health
|
||||
port: http
|
||||
# Mismo securityContext que Producción -- ver comentario largo en
|
||||
# workloads/ecommerce/frontend/patches/deployment.yaml (misma
|
||||
# imagen, mismos motivos: runAsUser numérico por el Dockerfile
|
||||
# usando "nextjs" por nombre, NET_BIND_SERVICE por el puerto 80,
|
||||
# volumenes por el cache de Next.js con root filesystem de solo
|
||||
# lectura).
|
||||
securityContext:
|
||||
allowPrivilegeEscalation: false
|
||||
readOnlyRootFilesystem: true
|
||||
capabilities:
|
||||
drop:
|
||||
- ALL
|
||||
add:
|
||||
- NET_BIND_SERVICE
|
||||
# Requests/limits mas chicos que Produccion -- una sola replica
|
||||
# de verificacion, no trafico real.
|
||||
resources:
|
||||
requests:
|
||||
cpu: 50m
|
||||
memory: 128Mi
|
||||
limits:
|
||||
cpu: 500m
|
||||
memory: 512Mi
|
||||
volumeMounts:
|
||||
- name: next-cache
|
||||
mountPath: /app/.next/cache
|
||||
- name: tmp
|
||||
mountPath: /tmp
|
||||
volumes:
|
||||
- name: next-cache
|
||||
emptyDir: {}
|
||||
- name: tmp
|
||||
emptyDir: {}
|
||||
@@ -0,0 +1,11 @@
|
||||
apiVersion: v1
|
||||
kind: Service
|
||||
metadata:
|
||||
name: app-svc
|
||||
spec:
|
||||
selector:
|
||||
app: frontend
|
||||
ports:
|
||||
- name: http
|
||||
port: 80
|
||||
targetPort: 80
|
||||
@@ -0,0 +1,19 @@
|
||||
apiVersion: networking.k8s.io/v1
|
||||
kind: Ingress
|
||||
metadata:
|
||||
name: ecommerce-staging-ingress
|
||||
annotations:
|
||||
traefik.ingress.kubernetes.io/router.entrypoints: web
|
||||
spec:
|
||||
ingressClassName: traefik
|
||||
rules:
|
||||
- host: staging.cruzcloud.net
|
||||
http:
|
||||
paths:
|
||||
- path: /
|
||||
pathType: Prefix
|
||||
backend:
|
||||
service:
|
||||
name: frontend-svc
|
||||
port:
|
||||
number: 80
|
||||
@@ -0,0 +1,8 @@
|
||||
apiVersion: kustomize.config.k8s.io/v1beta1
|
||||
kind: Kustomization
|
||||
|
||||
namespace: ecommerce-staging
|
||||
|
||||
resources:
|
||||
- frontend/
|
||||
- ingress-staging.yaml
|
||||
@@ -0,0 +1,57 @@
|
||||
import type { Metadata } from "next";
|
||||
import { MessageCircle, PackageOpen, ShieldCheck, Truck } from "lucide-react";
|
||||
import { Button } from "@/components/ui/button";
|
||||
|
||||
export const metadata: Metadata = { title: "Cambios | ARI Shopping" };
|
||||
|
||||
const WHATSAPP_URL = "https://wa.me/573016138155";
|
||||
|
||||
export default function CambiosPage() {
|
||||
return (
|
||||
<section className="mx-auto max-w-3xl px-4 py-14 lg:py-20">
|
||||
<span className="text-xs font-black uppercase tracking-[.18em] text-ari-coral">Ayuda</span>
|
||||
<h1 className="mt-3 text-4xl font-black tracking-tight sm:text-5xl">Cambios</h1>
|
||||
<p className="mt-5 text-lg leading-8 text-slate-600">
|
||||
Si tu pedido no es lo que esperabas, te ayudamos a cambiarlo.
|
||||
</p>
|
||||
|
||||
<div className="mt-10 grid gap-6 sm:grid-cols-2">
|
||||
<div className="rounded-2xl border border-slate-200 p-6">
|
||||
<PackageOpen className="size-6 text-ari-marino" />
|
||||
<h2 className="mt-3 font-black">5 días hábiles</h2>
|
||||
<p className="mt-2 text-sm leading-6 text-slate-600">
|
||||
Tenés 5 días hábiles desde que recibís tu pedido para solicitar un cambio.
|
||||
</p>
|
||||
</div>
|
||||
<div className="rounded-2xl border border-slate-200 p-6">
|
||||
<Truck className="size-6 text-ari-marino" />
|
||||
<h2 className="mt-3 font-black">Envío de vuelta sin costo</h2>
|
||||
<p className="mt-2 text-sm leading-6 text-slate-600">
|
||||
ARI Shopping asume el costo del envío de vuelta del producto — no tenés que pagar nada extra.
|
||||
</p>
|
||||
</div>
|
||||
<div className="rounded-2xl border border-slate-200 p-6">
|
||||
<ShieldCheck className="size-6 text-ari-marino" />
|
||||
<h2 className="mt-3 font-black">Condiciones</h2>
|
||||
<p className="mt-2 text-sm leading-6 text-slate-600">
|
||||
El producto debe estar sin usar, con su empaque y etiquetas originales.
|
||||
</p>
|
||||
</div>
|
||||
<div className="rounded-2xl border border-slate-200 p-6">
|
||||
<MessageCircle className="size-6 text-ari-marino" />
|
||||
<h2 className="mt-3 font-black">¿Cómo lo pido?</h2>
|
||||
<p className="mt-2 text-sm leading-6 text-slate-600">
|
||||
Escribinos por WhatsApp con tu número de pedido y coordinamos la recolección y el cambio.
|
||||
</p>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<Button asChild variant="primary" size="lg" className="mt-10">
|
||||
<a href={WHATSAPP_URL} target="_blank" rel="noopener noreferrer">
|
||||
<MessageCircle className="size-5" />
|
||||
Solicitar un cambio por WhatsApp
|
||||
</a>
|
||||
</Button>
|
||||
</section>
|
||||
);
|
||||
}
|
||||
@@ -0,0 +1,63 @@
|
||||
import type { Metadata } from "next";
|
||||
import { Mail, MapPin, MessageCircle } from "lucide-react";
|
||||
import { Button } from "@/components/ui/button";
|
||||
|
||||
export const metadata: Metadata = { title: "Contacto | ARI Shopping" };
|
||||
|
||||
// Numero real confirmado en vivo (probando el boton "Confirmar por
|
||||
// WhatsApp" del checkout con navegador real): mismo canal que ya usan
|
||||
// los clientes para confirmar pedidos, consistencia a proposito.
|
||||
const WHATSAPP_URL = "https://wa.me/573016138155";
|
||||
// Correo corporativo nuevo ([email protected] -> reenviado por
|
||||
// Cloudflare Email Routing a la casilla real) -- ver nota de
|
||||
// implementacion en ADR 0010 para el paso manual de DNS pendiente.
|
||||
const CONTACT_EMAIL = "[email protected]";
|
||||
|
||||
export default function ContactoPage() {
|
||||
return (
|
||||
<section className="mx-auto max-w-2xl px-4 py-14 text-center lg:py-20">
|
||||
<span className="text-xs font-black uppercase tracking-[.18em] text-ari-coral">Ayuda</span>
|
||||
<h1 className="mt-3 text-4xl font-black tracking-tight sm:text-5xl">Contacto</h1>
|
||||
<p className="mt-5 text-lg leading-8 text-slate-600">
|
||||
¿Tenés una pregunta sobre tu pedido, un producto o un cambio? Escribinos, te respondemos lo antes posible.
|
||||
</p>
|
||||
|
||||
<div className="mt-10 grid gap-4 sm:grid-cols-2">
|
||||
<a
|
||||
href={WHATSAPP_URL}
|
||||
target="_blank"
|
||||
rel="noopener noreferrer"
|
||||
className="flex flex-col items-center gap-3 rounded-2xl border border-slate-200 p-6 transition hover:border-ari-marino hover:bg-ari-marino/5"
|
||||
>
|
||||
<MessageCircle className="size-7 text-ari-marino" />
|
||||
<div>
|
||||
<p className="font-black">WhatsApp</p>
|
||||
<p className="mt-1 text-sm text-slate-500">+57 301 613 8155</p>
|
||||
</div>
|
||||
</a>
|
||||
<a
|
||||
href={`mailto:${CONTACT_EMAIL}`}
|
||||
className="flex flex-col items-center gap-3 rounded-2xl border border-slate-200 p-6 transition hover:border-ari-marino hover:bg-ari-marino/5"
|
||||
>
|
||||
<Mail className="size-7 text-ari-marino" />
|
||||
<div>
|
||||
<p className="font-black">Correo</p>
|
||||
<p className="mt-1 text-sm text-slate-500">{CONTACT_EMAIL}</p>
|
||||
</div>
|
||||
</a>
|
||||
</div>
|
||||
|
||||
<div className="mt-10 flex items-center justify-center gap-2 text-sm text-slate-500">
|
||||
<MapPin className="size-4" />
|
||||
Despachamos desde Santiago de Cali, Colombia
|
||||
</div>
|
||||
|
||||
<Button asChild variant="primary" size="lg" className="mt-10">
|
||||
<a href={WHATSAPP_URL} target="_blank" rel="noopener noreferrer">
|
||||
<MessageCircle className="size-5" />
|
||||
Escribirnos por WhatsApp
|
||||
</a>
|
||||
</Button>
|
||||
</section>
|
||||
);
|
||||
}
|
||||
@@ -0,0 +1,66 @@
|
||||
import type { Metadata } from "next";
|
||||
import { Calculator, Clock, MapPinned, Truck } from "lucide-react";
|
||||
|
||||
export const metadata: Metadata = { title: "Envíos | ARI Shopping" };
|
||||
|
||||
// ADR 0010 §2: el costo de envio se calcula en vivo con EnvioClick (peso +
|
||||
// direccion), no es un valor fijo -- el copy de abajo describe ese
|
||||
// comportamiento real, no un placeholder. Transportadoras reales
|
||||
// confirmadas en una cotizacion en vivo contra la API de EnvioClick
|
||||
// (PIBOX, COORDINADORA, ENVIA, RAPPI, TCC) -- EnvioClick cotiza entre
|
||||
// ellas y el sistema toma automaticamente la mas economica por pedido.
|
||||
const CARRIERS = ["Coordinadora", "Envía", "TCC", "Rappi", "Pibox"];
|
||||
|
||||
export default function EnviosPage() {
|
||||
return (
|
||||
<section className="mx-auto max-w-3xl px-4 py-14 lg:py-20">
|
||||
<span className="text-xs font-black uppercase tracking-[.18em] text-ari-coral">Ayuda</span>
|
||||
<h1 className="mt-3 text-4xl font-black tracking-tight sm:text-5xl">Envíos</h1>
|
||||
<p className="mt-5 text-lg leading-8 text-slate-600">
|
||||
Hacemos envíos a nivel nacional desde nuestra bodega en Santiago de Cali, Valle del Cauca
|
||||
</p>
|
||||
|
||||
<div className="mt-10 grid gap-6 sm:grid-cols-2">
|
||||
<div className="rounded-2xl border border-slate-200 p-6">
|
||||
<Calculator className="size-6 text-ari-marino" />
|
||||
<h2 className="mt-3 font-black">Costo calculado en tiempo real</h2>
|
||||
<p className="mt-2 text-sm leading-6 text-slate-600">
|
||||
El valor del envío se calcula automáticamente según el peso de tu pedido y la dirección de entrega —
|
||||
lo ves exacto en el checkout antes de pagar, nunca es una tarifa a ciegas.
|
||||
</p>
|
||||
</div>
|
||||
<div className="rounded-2xl border border-slate-200 p-6">
|
||||
<Truck className="size-6 text-ari-marino" />
|
||||
<h2 className="mt-3 font-black">Transportadoras aliadas</h2>
|
||||
<p className="mt-2 text-sm leading-6 text-slate-600">
|
||||
Cotizamos automáticamente entre {CARRIERS.join(", ")} y asignamos la tarifa más conveniente para cada
|
||||
pedido.
|
||||
</p>
|
||||
</div>
|
||||
<div className="rounded-2xl border border-slate-200 p-6">
|
||||
<Clock className="size-6 text-ari-marino" />
|
||||
<h2 className="mt-3 font-black">Tiempos de entrega</h2>
|
||||
<p className="mt-2 text-sm leading-6 text-slate-600">
|
||||
Varían según la transportadora asignada y tu ciudad — una vez despachado tu pedido te compartimos el
|
||||
número de guía para hacerle seguimiento.
|
||||
</p>
|
||||
</div>
|
||||
<div className="rounded-2xl border border-slate-200 p-6">
|
||||
<MapPinned className="size-6 text-ari-marino" />
|
||||
<h2 className="mt-3 font-black">Cobertura</h2>
|
||||
<p className="mt-2 text-sm leading-6 text-slate-600">
|
||||
Envíos a todo el territorio nacional colombiano desde nuestra bodega en Santiago de Cali, Valle del Cauca
|
||||
</p>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<p className="mt-10 text-sm text-slate-500">
|
||||
¿Tenés dudas puntuales sobre un pedido? Escribinos por{" "}
|
||||
<a href="/contacto" className="font-bold text-ari-marino hover:underline">
|
||||
nuestros canales de contacto
|
||||
</a>
|
||||
.
|
||||
</p>
|
||||
</section>
|
||||
);
|
||||
}
|
||||
@@ -0,0 +1,72 @@
|
||||
import type { Metadata } from "next";
|
||||
import Link from "next/link";
|
||||
import { Accordion, AccordionContent, AccordionItem, AccordionTrigger } from "@/components/ui/accordion";
|
||||
|
||||
export const metadata: Metadata = { title: "Preguntas frecuentes | ARI Shopping" };
|
||||
|
||||
// Solo preguntas cuya respuesta es un hecho real del storefront (ver
|
||||
// ADR 0009/0010) -- nada de metodos de pago, plazos u opciones que no
|
||||
// esten realmente implementados.
|
||||
const FAQS: { question: string; answer: React.ReactNode }[] = [
|
||||
{
|
||||
question: "¿Qué métodos de pago aceptan?",
|
||||
answer: "Pagás por PSE, con transferencia directa desde tu banco. Podés pagar como persona natural o jurídica.",
|
||||
},
|
||||
{
|
||||
question: "¿A qué ciudades hacen envíos?",
|
||||
answer: "A todo el territorio nacional colombiano. El costo del envío se calcula automáticamente según tu dirección y lo ves en el checkout antes de pagar.",
|
||||
},
|
||||
{
|
||||
question: "¿Cuánto tarda en llegar mi pedido?",
|
||||
answer: (
|
||||
<>
|
||||
Depende de la transportadora asignada y tu ciudad — más detalle en{" "}
|
||||
<Link href="/envios" className="font-bold text-ari-marino hover:underline">
|
||||
Envíos
|
||||
</Link>
|
||||
.
|
||||
</>
|
||||
),
|
||||
},
|
||||
{
|
||||
question: "¿Puedo cambiar un producto que no me sirvió?",
|
||||
answer: (
|
||||
<>
|
||||
Sí, tenés 5 días hábiles desde que lo recibís y nosotros asumimos el envío de vuelta — más detalle en{" "}
|
||||
<Link href="/cambios" className="font-bold text-ari-marino hover:underline">
|
||||
Cambios
|
||||
</Link>
|
||||
.
|
||||
</>
|
||||
),
|
||||
},
|
||||
{
|
||||
question: "Tengo otra pregunta, ¿cómo los contacto?",
|
||||
answer: (
|
||||
<>
|
||||
Escribinos por WhatsApp o correo desde{" "}
|
||||
<Link href="/contacto" className="font-bold text-ari-marino hover:underline">
|
||||
Contacto
|
||||
</Link>
|
||||
, te respondemos lo antes posible.
|
||||
</>
|
||||
),
|
||||
},
|
||||
];
|
||||
|
||||
export default function PreguntasFrecuentesPage() {
|
||||
return (
|
||||
<section className="mx-auto max-w-3xl px-4 py-14 lg:py-20">
|
||||
<span className="text-xs font-black uppercase tracking-[.18em] text-ari-coral">Ayuda</span>
|
||||
<h1 className="mt-3 text-4xl font-black tracking-tight sm:text-5xl">Preguntas frecuentes</h1>
|
||||
<Accordion type="single" collapsible className="mt-10">
|
||||
{FAQS.map((faq) => (
|
||||
<AccordionItem key={faq.question} value={faq.question}>
|
||||
<AccordionTrigger>{faq.question}</AccordionTrigger>
|
||||
<AccordionContent>{faq.answer}</AccordionContent>
|
||||
</AccordionItem>
|
||||
))}
|
||||
</Accordion>
|
||||
</section>
|
||||
);
|
||||
}
|
||||
@@ -1,7 +1,7 @@
|
||||
import type { Metadata } from "next";
|
||||
import { notFound } from "next/navigation";
|
||||
import { Accordion, AccordionContent, AccordionItem, AccordionTrigger } from "@/components/ui/accordion";
|
||||
import { MobileAddToCart } from "@/components/product/mobile-add-to-cart";
|
||||
import { ProductCard } from "@/components/product/product-card";
|
||||
import { ProductCarousel } from "@/components/product/product-carousel";
|
||||
import { ProductGallery } from "@/components/product/product-gallery";
|
||||
import { ProductPurchase } from "@/components/product/product-purchase";
|
||||
@@ -24,7 +24,17 @@ export const dynamic = "force-dynamic";
|
||||
export async function generateMetadata({ params }: ProductPageProps): Promise<Metadata> {
|
||||
const { slug } = await params;
|
||||
const product = await getProductBySlug(slug);
|
||||
return product ? { title: product.name, description: product.shortDescription, openGraph: { images: [product.image] } } : { title: "Producto" };
|
||||
// notFound() se llama aquí (no solo en el componente de abajo) porque
|
||||
// generateMetadata se resuelve por completo ANTES de que Next empiece a
|
||||
// transmitir la respuesta -- el <Suspense> del navbar (categorías, ver
|
||||
// navbar.tsx) hace que el layout raíz empiece a stream-ear con status 200
|
||||
// antes de que un notFound() lanzado dentro del componente de la página
|
||||
// pueda cambiarlo (confirmado en vivo: cualquier slug inexistente devolvía
|
||||
// HTTP 200 con el contenido de app/not-found.tsx, rompiendo SEO/crawlers
|
||||
// que sí miran el status code). Llamarlo aquí evita el problema porque esta
|
||||
// fase bloquea el stream por diseño.
|
||||
if (!product) notFound();
|
||||
return { title: product.name, description: product.shortDescription, openGraph: { images: [product.image] } };
|
||||
}
|
||||
|
||||
export default async function ProductPage({ params }: ProductPageProps) {
|
||||
@@ -33,5 +43,84 @@ export default async function ProductPage({ params }: ProductPageProps) {
|
||||
if (!product) notFound();
|
||||
const products = await getProducts();
|
||||
const recommended = products.filter((candidate) => candidate.id !== product.id && (candidate.category === product.category || candidate.collection === product.collection)).slice(0, 8);
|
||||
return <><section className="mx-auto grid max-w-7xl gap-10 px-4 py-10 lg:grid-cols-[1.08fr_.92fr] lg:py-16"><ProductGallery images={product.images} name={product.name} /><div><span className="text-xs font-black uppercase tracking-[.18em] text-ari-coral">{product.category} · {product.collection}</span><h1 className="mt-3 text-4xl font-black tracking-tight sm:text-6xl">{product.name}</h1><p className="mt-5 text-lg leading-8 text-slate-600">{product.description}</p><div className="mt-8"><ProductPurchase product={product} /></div><Accordion type="single" collapsible className="mt-8"><AccordionItem value="details"><AccordionTrigger>Detalles del producto</AccordionTrigger><AccordionContent>Marca: {product.brand}. Edad recomendada: {product.ageRange}. La disponibilidad y presentación final se confirman con Ari Shopping.</AccordionContent></AccordionItem><AccordionItem value="shipping"><AccordionTrigger>Envíos y entregas</AccordionTrigger><AccordionContent>Coordinamos envíos nacionales y entrega según ubicación. Los tiempos se confirman durante la cotización.</AccordionContent></AccordionItem></Accordion></div></section>{recommended.length > 0 && <section className="mx-auto max-w-7xl px-4 pb-24"><h2 className="mb-7 text-3xl font-black">También te puede gustar</h2><ProductCarousel products={recommended} /></section>}<MobileAddToCart product={product} /></>;
|
||||
// Categoría distinta a propósito -- antes repetía los primeros 2 del
|
||||
// mismo "recommended" que ya alimenta el carrusel de abajo (mismos
|
||||
// productos en dos lugares de la página, reportado por el usuario).
|
||||
// Un cross-sell real de otra categoría aporta algo que el carrusel no
|
||||
// muestra, en vez de duplicarlo.
|
||||
const crossCategory = products.filter((candidate) => candidate.id !== product.id && candidate.category !== product.category);
|
||||
const crossSell = crossCategory.find((candidate) => candidate.available) ?? crossCategory[0];
|
||||
|
||||
return (
|
||||
<>
|
||||
{/* max-w-[1600px] (antes max-w-7xl/1280px) y una 3ra columna desde
|
||||
xl -- aprovecha pantallas grandes en vez de dejar el buy box
|
||||
apretado contra el texto o cortado por el panel de zoom de al
|
||||
lado (ver ProductGallery). Por debajo de xl, el buy box cae al
|
||||
flujo normal en la 2da columna, sin duplicar el componente en
|
||||
el DOM.
|
||||
El buy box ocupa las 2 filas (xl:row-span-2) para no quedar
|
||||
atado a la altura de galería+descripción -- "También te puede
|
||||
gustar" pasa a ser la 2da fila debajo de esas dos columnas
|
||||
(xl:col-span-2), en paralelo al buy box sticky, en vez de
|
||||
esperar a que termine la columna más alta antes de aparecer. */}
|
||||
<section className="mx-auto grid max-w-[1600px] gap-x-10 gap-y-12 px-4 py-10 lg:grid-cols-[1.08fr_.92fr] lg:py-16 xl:grid-cols-[1.1fr_.75fr_360px] xl:grid-rows-[auto_auto]">
|
||||
<ProductGallery images={product.images} name={product.name} />
|
||||
<div className="min-w-0">
|
||||
<span className="text-xs font-black uppercase tracking-[.18em] text-ari-coral">{product.category} · {product.collection}</span>
|
||||
<h1 className="mt-3 text-4xl font-black tracking-tight sm:text-6xl">{product.name}</h1>
|
||||
<p className="mt-5 text-sm leading-6 text-slate-600">{product.description}</p>
|
||||
{/* Antes escondido detrás de un click en el acordeón --
|
||||
siempre visible, estilo Amazon, con los mismos datos
|
||||
reales que ya existían (marca/edad/categoría/colección),
|
||||
sin inventar specs que el catálogo no tiene. */}
|
||||
<h2 className="mb-4 mt-8 text-sm font-black uppercase tracking-wide text-slate-500">Detalles del producto</h2>
|
||||
<dl className="grid grid-cols-2 gap-x-6 gap-y-4 rounded-2xl border border-slate-200 p-5 text-sm sm:grid-cols-4">
|
||||
<div><dt className="text-slate-400">Marca</dt><dd className="mt-0.5 font-bold text-slate-900">{product.brand}</dd></div>
|
||||
<div><dt className="text-slate-400">Edad recomendada</dt><dd className="mt-0.5 font-bold text-slate-900">{product.ageRange}</dd></div>
|
||||
<div><dt className="text-slate-400">Categoría</dt><dd className="mt-0.5 font-bold text-slate-900">{product.category}</dd></div>
|
||||
<div><dt className="text-slate-400">Colección</dt><dd className="mt-0.5 font-bold text-slate-900">{product.collection}</dd></div>
|
||||
</dl>
|
||||
<Accordion type="single" collapsible className="mt-6">
|
||||
<AccordionItem value="shipping">
|
||||
<AccordionTrigger>Envíos y entregas</AccordionTrigger>
|
||||
<AccordionContent>Coordinamos envíos nacionales y entrega según ubicación. Los tiempos se confirman durante la cotización.</AccordionContent>
|
||||
</AccordionItem>
|
||||
</Accordion>
|
||||
</div>
|
||||
{/* Buy box estilo Amazon: precio/variantes/carrito en su propia
|
||||
tarjeta, sticky desde xl mientras el resto de la pagina
|
||||
scrollea. El cross-sell es de OTRA categoría a propósito --
|
||||
evita repetir lo que ya muestra el carrusel de "También te
|
||||
puede gustar" de abajo (mismo criterio de categoría/colección
|
||||
que ese carrusel), aportando un producto genuinamente distinto
|
||||
en vez de duplicado. */}
|
||||
<div className="min-w-0 xl:sticky xl:top-24 xl:row-span-2 xl:self-start">
|
||||
<div className="rounded-2xl border border-slate-200 p-6 shadow-sm">
|
||||
<ProductPurchase product={product} />
|
||||
</div>
|
||||
{crossSell && (
|
||||
<div className="mt-6">
|
||||
<h2 className="mb-4 text-xs font-black uppercase tracking-[.14em] text-slate-500">También te puede interesar</h2>
|
||||
<ProductCard product={crossSell} />
|
||||
</div>
|
||||
)}
|
||||
</div>
|
||||
{/* min-w-0 es necesario: sin esto, el carrusel horizontal
|
||||
(overflow-x-auto interno) contaba como item de este grid, y su
|
||||
ancho minimo intrinseco (la suma de TODAS las cards sin
|
||||
wrappear, ya que el scroller no tiene flex-wrap) empujaba todo
|
||||
el grid completo a ~800px en mobile -- afectaba a TODA la
|
||||
columna compartida (imagen, titulo, specs), no solo al
|
||||
carrusel. Confirmado en vivo: sin min-w-0 el grid medía 783px
|
||||
en un viewport real de 380px; con min-w-0 vuelve a 380px. */}
|
||||
{recommended.length > 0 && (
|
||||
<div className="min-w-0 pb-16 lg:col-span-2">
|
||||
<h2 className="mb-7 text-3xl font-black">También te puede gustar</h2>
|
||||
<ProductCarousel products={recommended} />
|
||||
</div>
|
||||
)}
|
||||
</section>
|
||||
</>
|
||||
);
|
||||
}
|
||||
@@ -38,12 +38,13 @@ data:
|
||||
# reemplaza la tarifa fija. Solo config no sensible; ENVIOCLICK_API_KEY
|
||||
# va en secrets.template.yaml. ENVIOCLICK_SANDBOX=true usa el endpoint
|
||||
# de guía de pruebas (/shipment_sandbox) -- la cotización siempre usa
|
||||
# tarifas reales, no tiene modo sandbox. Origen = bodega Bogotá
|
||||
# ("Bodega principal ARI Shopping", sin dirección física real cargada
|
||||
# todavía -- ENVIOCLICK_ORIGIN_ADDRESS es un placeholder hasta que la
|
||||
# haya).
|
||||
# tarifas reales, no tiene modo sandbox. Origen = bodega Santiago de
|
||||
# Cali, Valle del Cauca ("Bodega principal ARI Shopping") -- corregido
|
||||
# 2026-09-12: el código DANE cargado originalmente era el de Bogotá
|
||||
# (11001000) por error; toda cotización real hecha antes de esta
|
||||
# fecha calculó la tarifa desde el origen equivocado.
|
||||
ENVIOCLICK_SANDBOX: "true"
|
||||
ENVIOCLICK_ORIGIN_DANE_CODE: "11001000"
|
||||
ENVIOCLICK_ORIGIN_DANE_CODE: "76001000"
|
||||
ENVIOCLICK_ORIGIN_ADDRESS: "Cra 100 #42-65"
|
||||
ENVIOCLICK_ORIGIN_COMPANY: "ARI Shopping"
|
||||
ENVIOCLICK_ORIGIN_EMAIL: "[email protected]"
|
||||
|
||||
@@ -65,7 +65,7 @@ spec:
|
||||
memory: 64Mi
|
||||
|
||||
- name: migrations
|
||||
image: gitea.cruzcloud.net/devops/ecommerce-medusa:v1.0.234
|
||||
image: gitea.cruzcloud.net/devops/ecommerce-medusa:v1.0.265
|
||||
imagePullPolicy: IfNotPresent
|
||||
command:
|
||||
- node
|
||||
@@ -86,7 +86,7 @@ spec:
|
||||
|
||||
containers:
|
||||
- name: app
|
||||
image: gitea.cruzcloud.net/devops/ecommerce-medusa:v1.0.234
|
||||
image: gitea.cruzcloud.net/devops/ecommerce-medusa:v1.0.265
|
||||
imagePullPolicy: IfNotPresent
|
||||
envFrom:
|
||||
- configMapRef:
|
||||
|
||||
@@ -1,10 +1,51 @@
|
||||
import Link from "next/link";
|
||||
import { Facebook, Instagram, MessageCircle } from "lucide-react";
|
||||
|
||||
// APP_VERSION la actualiza el propio CI (.gitea/workflows/build.yaml) al
|
||||
// mismo tiempo que el tag de la imagen -- antes decia "v4.1" fijo,
|
||||
// inutil para distinguir a simple vista si Staging y Produccion estan
|
||||
// en la misma version o no.
|
||||
const appVersion = process.env.APP_VERSION || "dev";
|
||||
|
||||
// Las 4 paginas de "Ayuda" ya tienen contenido real (app/envios,
|
||||
// app/cambios, app/preguntas-frecuentes, app/contacto).
|
||||
const HELP_LINKS = [
|
||||
{ label: "Preguntas frecuentes", href: "/preguntas-frecuentes" },
|
||||
{ label: "Envíos", href: "/envios" },
|
||||
{ label: "Cambios", href: "/cambios" },
|
||||
{ label: "Contacto", href: "/contacto" },
|
||||
];
|
||||
|
||||
// Cuentas reales confirmadas por el usuario (Instagram verificado en
|
||||
// vivo antes de linkearlo -- la primera cuenta sugerida, @arishopping,
|
||||
// no era la real). El link de WhatsApp usa el mismo numero real ya
|
||||
// confirmado en /contacto y en el fallback del checkout.
|
||||
const SOCIAL_LINKS = [
|
||||
{ label: "Instagram", href: "https://www.instagram.com/ari_shopping1/", Icon: Instagram },
|
||||
{ label: "Facebook", href: "https://www.facebook.com/share/1Dkmvko6HF/", Icon: Facebook },
|
||||
{ label: "WhatsApp", href: "https://wa.me/573016138155", Icon: MessageCircle },
|
||||
];
|
||||
|
||||
const SHOP_LINKS = [
|
||||
{ label: "Novedades", href: "/catalog" },
|
||||
{ label: "Juguetes", href: "/catalog" },
|
||||
{ label: "Mochilas", href: "/catalog" },
|
||||
{ label: "Regalos", href: "/catalog" },
|
||||
];
|
||||
|
||||
export function Footer() {
|
||||
return <footer className="bg-ari-marino text-white"><div className="mx-auto grid max-w-7xl gap-10 px-4 py-14 sm:grid-cols-2 lg:grid-cols-4"><div><div className="flex flex-col items-start gap-3"><img src="/images/brand/logo-ari.svg" alt="ARI shopping" className="h-11 w-auto" width={200} height={64} />{/* eslint-disable-line @next/next/no-img-element -- vector propio, next/image no optimiza SVG sin habilitar dangerouslyAllowSVG */}<p className="text-xs text-white/60">Tesoros para cada aventura.</p></div><div className="mt-5 flex gap-3"><Instagram /><Facebook /><MessageCircle /></div></div><FooterColumn title="Compra" links={["Novedades", "Juguetes", "Mochilas", "Regalos"]} /><FooterColumn title="Ayuda" links={["Preguntas frecuentes", "Envíos", "Cambios", "Contacto"]} /><div><h3 className="font-black">Newsletter</h3><p className="mt-3 text-sm leading-6 text-white/60">Recibe lanzamientos y promociones especiales.</p><div className="mt-4 flex rounded-full bg-white p-1"><input className="min-w-0 flex-1 bg-transparent px-4 text-sm text-slate-950 outline-none" placeholder="Tu correo" /><button className="rounded-full bg-ari-coral px-4 text-sm font-black text-white">Unirme</button></div></div></div><div className="border-t border-white/10 py-5 text-center text-xs text-white/50">© 2026 ARI Shopping · Frontend headless v4.1 · GitOps</div></footer>;
|
||||
return <footer className="bg-ari-marino text-white"><div className="mx-auto grid max-w-7xl gap-10 px-4 py-14 sm:grid-cols-2 lg:grid-cols-4"><div><div className="flex flex-col items-start gap-3"><img src="/images/brand/logo-ari.svg" alt="ARI shopping" className="h-11 w-auto" width={200} height={64} />{/* eslint-disable-line @next/next/no-img-element -- vector propio, next/image no optimiza SVG sin habilitar dangerouslyAllowSVG */}<p className="text-xs text-white/60">Tesoros para cada aventura.</p></div><div className="mt-5 flex gap-3">{SOCIAL_LINKS.map(({ label, href, Icon }) => <a key={label} href={href} target="_blank" rel="noopener noreferrer" aria-label={label} className="text-white/80 transition hover:text-ari-coral-claro"><Icon /></a>)}</div></div><FooterColumn title="Compra" links={SHOP_LINKS} /><FooterColumn title="Ayuda" links={HELP_LINKS} />{/* min-w-0: sin esto, un grid item de texto toma como minimo su ancho
|
||||
max-content (una sola linea, "Recibe lanzamientos y promociones
|
||||
especiales." sin wrappear, ~335px) en vez de dejarse achicar y
|
||||
wrappear dentro del track del grid -- confirmado en vivo 2026-09-20
|
||||
contra un viewport real de 320px: la columna entera del footer (y
|
||||
con ella, la pagina completa) quedaba forzada a 335px de ancho por
|
||||
este unico parrafo. Mismo bug de fondo que el ya documentado en
|
||||
product-gallery.tsx/app/product/[slug]/page.tsx, en un grid en vez
|
||||
de un overflow-x-auto. */}
|
||||
<div className="min-w-0"><h3 className="font-black">Newsletter</h3><p className="mt-3 text-sm leading-6 text-white/60">Recibe lanzamientos y promociones especiales.</p><div className="mt-4 flex rounded-full bg-white p-1"><input className="min-w-0 flex-1 bg-transparent px-4 text-sm text-slate-950 outline-none" placeholder="Tu correo" /><button className="rounded-full bg-ari-coral px-4 text-sm font-black text-white">Unirme</button></div></div></div><div className="border-t border-white/10 py-5 text-center text-xs text-white/50">© 2026 ARI Shopping · Frontend headless {appVersion} · GitOps</div></footer>;
|
||||
}
|
||||
|
||||
function FooterColumn({ title, links }: { title: string; links: string[] }) {
|
||||
return <div><h3 className="font-black">{title}</h3><div className="mt-4 grid gap-3 text-sm text-white/60">{links.map((link) => <Link key={link} href="/catalog" className="hover:text-ari-coral-claro">{link}</Link>)}</div></div>;
|
||||
function FooterColumn({ title, links }: { title: string; links: { label: string; href: string }[] }) {
|
||||
return <div><h3 className="font-black">{title}</h3><div className="mt-4 grid gap-3 text-sm text-white/60">{links.map((link) => <Link key={link.label} href={link.href} className="hover:text-ari-coral-claro">{link.label}</Link>)}</div></div>;
|
||||
}
|
||||
@@ -261,10 +261,18 @@ function DepartmentsMegaMenu({ tree, isActive }: { tree: CategoryNode[]; isActiv
|
||||
onClick={() => setOpen((value) => !value)}
|
||||
aria-expanded={open}
|
||||
aria-haspopup="true"
|
||||
aria-label="Departamentos"
|
||||
className={`flex items-center gap-1.5 ${pillClass(open)}`}
|
||||
>
|
||||
<LayoutGrid className="size-4" />
|
||||
Departamentos
|
||||
{/* Texto oculto en mobile: los 3 items de esta fila (menu, este
|
||||
boton, "Todo") tienen shrink-0 -- ninguno se achica, asi que si
|
||||
los tres no entran, desbordan la pagina entera (confirmado en
|
||||
vivo 2026-09-20 contra 320px real: 14px de overflow horizontal
|
||||
con el texto visible). No es un caso de min-w-0 faltante (ese
|
||||
fix asume que el item PUEDE achicarse) -- ocultar la palabra
|
||||
mas larga de la fila es lo que de verdad libera espacio. */}
|
||||
<span className="hidden sm:inline">Departamentos</span>
|
||||
<ChevronDown className={`size-3.5 opacity-70 transition-transform ${open ? "rotate-180" : ""}`} />
|
||||
</button>
|
||||
|
||||
|
||||
@@ -1,25 +0,0 @@
|
||||
"use client";
|
||||
|
||||
import { ShoppingBag } from "lucide-react";
|
||||
import { Button } from "@/components/ui/button";
|
||||
import type { Product } from "@/lib/headless/types";
|
||||
import { useCartStore } from "@/store/cart-store";
|
||||
|
||||
export function MobileAddToCart({ product }: { product: Product }) {
|
||||
const addItem = useCartStore((state) => state.addItem);
|
||||
const outOfStock = product.variants[0]?.available === false;
|
||||
return (
|
||||
<div className="fixed inset-x-0 bottom-0 z-40 border-t border-slate-200 bg-white/95 p-3 shadow-2xl backdrop-blur lg:hidden">
|
||||
<Button
|
||||
variant="primary"
|
||||
size="lg"
|
||||
className="w-full"
|
||||
disabled={outOfStock}
|
||||
onClick={() => addItem(product, product.variants[0]?.id)}
|
||||
>
|
||||
<ShoppingBag />
|
||||
{outOfStock ? "Agotado" : "Añadir al carrito"}
|
||||
</Button>
|
||||
</div>
|
||||
);
|
||||
}
|
||||
@@ -7,28 +7,82 @@ import { Dialog, DialogContent, DialogTrigger } from "@/components/ui/dialog";
|
||||
|
||||
interface Offset { x: number; y: number }
|
||||
interface DragState { pointerId: number; startX: number; startY: number; originX: number; originY: number }
|
||||
interface HoverZoomOrigin { x: number; y: number }
|
||||
interface HoverZoomState {
|
||||
lensLeft: number;
|
||||
lensTop: number;
|
||||
lensSize: number;
|
||||
bgX: number;
|
||||
bgY: number;
|
||||
showPanel: boolean;
|
||||
panelLeft: number;
|
||||
panelTop: number;
|
||||
panelSize: number;
|
||||
}
|
||||
|
||||
// Zoom automatico al pasar el cursor (estilo lupa clasico de e-commerce)
|
||||
// sobre la imagen principal -- solo mouse (mousemove no dispara de forma
|
||||
// continua en touch, asi que en mobile simplemente no aparece el efecto,
|
||||
// sin romper nada). Independiente del zoom interactivo del modal
|
||||
// (click -> Dialog con scale/drag), que sigue igual.
|
||||
// Zoom estilo "lupa" de Amazon: un recuadro semitransparente sobre la
|
||||
// imagen (la lupa) marca la region que se esta mirando, y un panel
|
||||
// aparte a la derecha muestra esa misma region ampliada -- sin recortar
|
||||
// nada de la pantalla ni pedirle al usuario que haga scroll para verlo
|
||||
// completo (a diferencia de un primer intento que escalaba la imagen
|
||||
// in-place con transform, donde para ver el borde superior había que
|
||||
// mover el cursor fuera del viewport visible). Solo mouse (mousemove no
|
||||
// dispara de forma continua en touch); en mobile no aparece el efecto,
|
||||
// sin romper nada. Independiente del zoom interactivo del modal (click
|
||||
// -> Dialog con scale/drag), que sigue igual.
|
||||
//
|
||||
// El panel usa background-image con la foto completa (no la version ya
|
||||
// recortada por object-cover) a proposito -- escalar la imagen recortada
|
||||
// nunca iba a revelar las partes recortadas (ej. la cabeza de una figura
|
||||
// de accion en una foto vertical). background-size en un solo valor
|
||||
// (ancho%, alto en "auto") preserva el aspect ratio real de la foto
|
||||
// completa.
|
||||
//
|
||||
// El tamaño/posicion del panel se calculan en cada mousemove contra el
|
||||
// viewport real (window.innerWidth/innerHeight), no con un breakpoint
|
||||
// fijo tipo "lg:block" -- a resoluciones altas con la ventana angosta
|
||||
// (o el layout de la pagina dejando poco espacio a la derecha de la
|
||||
// imagen) un tamaño fijo terminaba saliéndose de la pantalla. Si no
|
||||
// entra un panel de al menos PANEL_MIN_SIZE, se cae automáticamente al
|
||||
// overlay in-place (misma imagen completa como fondo, dentro de la
|
||||
// propia caja) en vez de mostrar un panel cortado.
|
||||
const HOVER_ZOOM_SCALE = 2.2;
|
||||
const LENS_RATIO = 1 / HOVER_ZOOM_SCALE;
|
||||
const PANEL_MARGIN = 16;
|
||||
const PANEL_MAX_SIZE = 480;
|
||||
const PANEL_MIN_SIZE = 220;
|
||||
const VIEWPORT_EDGE_PADDING = 16;
|
||||
|
||||
export function ProductGallery({ images, name }: { images: string[]; name: string }) {
|
||||
const [selected, setSelected] = useState(images[0] ?? "");
|
||||
const [open, setOpen] = useState(false);
|
||||
const [scale, setScale] = useState(1);
|
||||
const [offset, setOffset] = useState<Offset>({ x: 0, y: 0 });
|
||||
const [hoverZoom, setHoverZoom] = useState<HoverZoomOrigin | null>(null);
|
||||
const [hoverZoom, setHoverZoom] = useState<HoverZoomState | null>(null);
|
||||
const drag = useRef<DragState | null>(null);
|
||||
|
||||
const trackHoverZoom = (event: MouseEvent<HTMLButtonElement>) => {
|
||||
const rect = event.currentTarget.getBoundingClientRect();
|
||||
const lensSize = rect.width * LENS_RATIO;
|
||||
const cursorX = event.clientX - rect.left;
|
||||
const cursorY = event.clientY - rect.top;
|
||||
const lensLeft = Math.min(Math.max(cursorX - lensSize / 2, 0), rect.width - lensSize);
|
||||
const lensTop = Math.min(Math.max(cursorY - lensSize / 2, 0), rect.height - lensSize);
|
||||
|
||||
const availableWidth = window.innerWidth - rect.right - PANEL_MARGIN - VIEWPORT_EDGE_PADDING;
|
||||
const availableHeight = window.innerHeight - VIEWPORT_EDGE_PADDING * 2;
|
||||
const panelSize = Math.min(PANEL_MAX_SIZE, availableWidth, availableHeight);
|
||||
const panelTop = Math.min(Math.max(rect.top, VIEWPORT_EDGE_PADDING), window.innerHeight - panelSize - VIEWPORT_EDGE_PADDING);
|
||||
|
||||
setHoverZoom({
|
||||
x: ((event.clientX - rect.left) / rect.width) * 100,
|
||||
y: ((event.clientY - rect.top) / rect.height) * 100,
|
||||
lensLeft,
|
||||
lensTop,
|
||||
lensSize,
|
||||
bgX: (lensLeft / (rect.width - lensSize)) * 100,
|
||||
bgY: (lensTop / (rect.height - lensSize)) * 100,
|
||||
showPanel: panelSize >= PANEL_MIN_SIZE,
|
||||
panelLeft: rect.right + PANEL_MARGIN,
|
||||
panelTop,
|
||||
panelSize,
|
||||
});
|
||||
};
|
||||
|
||||
@@ -55,38 +109,101 @@ export function ProductGallery({ images, name }: { images: string[]; name: strin
|
||||
};
|
||||
|
||||
return (
|
||||
<div className="grid gap-4 lg:grid-cols-[86px_1fr]">
|
||||
<div className="min-w-0 grid gap-4 lg:grid-cols-[86px_1fr]">
|
||||
{/* min-w-0: esta tira de miniaturas es un overflow-x-auto (scroll
|
||||
horizontal) igual que el carrusel de "También te puede gustar"
|
||||
en app/product/[slug]/page.tsx -- mismo bug si este componente
|
||||
termina como item de un grid externo sin min-w-0: para productos
|
||||
con muchas fotos, el ancho minimo intrinseco de TODAS las
|
||||
miniaturas sin wrappear puede forzar overflow horizontal en toda
|
||||
la pagina en mobile. */}
|
||||
<div className="order-2 flex gap-3 overflow-x-auto lg:order-1 lg:flex-col">
|
||||
{images.map((image) => <button type="button" key={image} onClick={() => { setSelected(image); reset(); }} className={`relative aspect-square w-20 shrink-0 overflow-hidden rounded-2xl border-2 ${selected === image ? "border-fuchsia-500" : "border-transparent"}`}><Image src={image} alt="" fill sizes="80px" className="object-cover" /></button>)}
|
||||
{/* object-contain (no object-cover): fotos de producto con relacion
|
||||
de aspecto muy vertical (ej. 933x1500, confirmado en vivo para
|
||||
este catalogo) perdian la cabeza o los pies recortados al forzar
|
||||
el cuadrado con cover -- reportado por el usuario viendo las
|
||||
miniaturas. contain deja la miniatura completa siempre, con
|
||||
"letterbox" del mismo bg-slate-50 que usa la imagen principal. */}
|
||||
{images.map((image) => <button type="button" key={image} onClick={() => { setSelected(image); reset(); }} onMouseEnter={() => { setSelected(image); reset(); }} className={`relative aspect-square w-20 shrink-0 overflow-hidden rounded-2xl border-2 bg-slate-50 ${selected === image ? "border-fuchsia-500" : "border-transparent"}`}><Image src={image} alt="" fill sizes="80px" className="object-contain" /></button>)}
|
||||
</div>
|
||||
<Dialog open={open} onOpenChange={changeOpen}>
|
||||
<DialogTrigger asChild>
|
||||
<button
|
||||
type="button"
|
||||
className="group relative order-1 aspect-square cursor-zoom-in overflow-hidden rounded-[2rem] bg-slate-50 lg:order-2"
|
||||
onMouseMove={trackHoverZoom}
|
||||
onMouseLeave={() => setHoverZoom(null)}
|
||||
>
|
||||
<Image
|
||||
src={selected}
|
||||
alt={name}
|
||||
fill
|
||||
priority
|
||||
sizes="(max-width: 1024px) 100vw, 50vw"
|
||||
className="object-cover transition-transform duration-150 ease-out"
|
||||
style={{
|
||||
transform: `scale(${hoverZoom ? HOVER_ZOOM_SCALE : 1})`,
|
||||
transformOrigin: hoverZoom ? `${hoverZoom.x}% ${hoverZoom.y}%` : "center",
|
||||
}}
|
||||
/>
|
||||
<span className="absolute bottom-5 right-5 inline-flex items-center gap-2 rounded-full bg-white px-4 py-2 text-xs font-black shadow-xl"><ZoomIn className="size-4" />Zoom interactivo</span>
|
||||
</button>
|
||||
</DialogTrigger>
|
||||
<DialogContent className="max-w-6xl overflow-hidden bg-slate-950 p-0">
|
||||
<div className="relative order-1 lg:order-2">
|
||||
<Dialog open={open} onOpenChange={changeOpen}>
|
||||
<DialogTrigger asChild>
|
||||
<button
|
||||
type="button"
|
||||
className="group relative aspect-square w-full cursor-zoom-in overflow-hidden rounded-[2rem] bg-slate-50"
|
||||
onMouseMove={trackHoverZoom}
|
||||
onMouseLeave={() => setHoverZoom(null)}
|
||||
>
|
||||
{/* unoptimized a proposito: confirmado en vivo (createImageBitmap
|
||||
sobre los bytes reales vs. el <img> renderizado) que a DPR
|
||||
alto (2.6+, cualquier Android real) Chromium reporta
|
||||
naturalWidth/Height de un candidato del srcset mucho más
|
||||
chico del que dice currentSrc -- bug del propio motor de
|
||||
Chromium en la selección de candidato con sizes="100vw" en
|
||||
fill, no algo corrupto en el archivo ni en nuestro caché
|
||||
(el archivo decodifica perfecto vía fetch+createImageBitmap
|
||||
fuera de <img>). Reproducido igual en Chrome y Edge Android
|
||||
(ambos Chromium) y en Chromium headless. unoptimized deja
|
||||
un solo recurso real por foto -- sin candidatos entre los
|
||||
que el bug pueda elegir mal. */}
|
||||
{/* object-contain (no object-cover): mismo motivo que las
|
||||
miniaturas -- fotos verticales (933x1500 confirmado en
|
||||
este catalogo) perdian la cabeza al forzar el cuadrado.
|
||||
Reportado en vivo: al seleccionar esa foto, Buzz aparecia
|
||||
sin cabeza en la imagen principal. */}
|
||||
<Image
|
||||
src={selected}
|
||||
alt={name}
|
||||
fill
|
||||
priority
|
||||
unoptimized
|
||||
className="object-contain"
|
||||
/>
|
||||
{/* Lupa: recuadro que marca la region ampliada -- solo cuando
|
||||
hay lugar real para el panel de al lado (showPanel,
|
||||
calculado contra el viewport real, no un breakpoint
|
||||
fijo). Sin espacio para el panel se usa el overlay
|
||||
in-place como fallback, para cualquier resolucion. */}
|
||||
{hoverZoom?.showPanel && (
|
||||
<div
|
||||
aria-hidden
|
||||
data-testid="gallery-lens"
|
||||
className="pointer-events-none absolute border-2 border-white bg-white/25 shadow-lg"
|
||||
style={{ left: hoverZoom.lensLeft, top: hoverZoom.lensTop, width: hoverZoom.lensSize, height: hoverZoom.lensSize }}
|
||||
/>
|
||||
)}
|
||||
<div
|
||||
aria-hidden
|
||||
data-testid="gallery-inplace-overlay"
|
||||
className="pointer-events-none absolute inset-0 bg-slate-50 transition-opacity duration-150 ease-out"
|
||||
style={{
|
||||
opacity: hoverZoom && !hoverZoom.showPanel ? 1 : 0,
|
||||
// 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",
|
||||
backgroundSize: `${HOVER_ZOOM_SCALE * 100}%`,
|
||||
backgroundPosition: hoverZoom ? `${hoverZoom.bgX}% ${hoverZoom.bgY}%` : "center",
|
||||
}}
|
||||
/>
|
||||
<span className="absolute bottom-5 right-5 inline-flex items-center gap-2 rounded-full bg-white px-4 py-2 text-xs font-black shadow-xl"><ZoomIn className="size-4" />Zoom interactivo</span>
|
||||
</button>
|
||||
</DialogTrigger>
|
||||
<DialogContent className="max-w-6xl overflow-hidden bg-slate-950 p-0">
|
||||
<div className="grid min-h-[70vh] grid-rows-[1fr_auto]">
|
||||
<div className="relative overflow-hidden bg-slate-950" onWheel={wheel} onPointerDown={pointerDown} onPointerMove={pointerMove} onPointerUp={pointerEnd} onPointerCancel={pointerEnd} onDoubleClick={() => applyScale(scale > 1 ? 1 : 2)} style={{ touchAction: "none", cursor: scale > 1 ? "grab" : "zoom-in" }}>
|
||||
<div className="absolute inset-6 transition-transform duration-100 will-change-transform" style={{ transform: `translate3d(${offset.x}px, ${offset.y}px, 0) scale(${scale})` }}>
|
||||
<Image src={selected} alt={name} fill sizes="95vw" className="select-none object-contain" draggable={false} />
|
||||
<Image src={selected} alt={name} fill unoptimized className="select-none object-contain" draggable={false} />
|
||||
</div>
|
||||
<div className="pointer-events-none absolute left-5 top-5 rounded-full bg-black/65 px-3 py-1.5 text-xs font-black text-white backdrop-blur">Rueda o botones para acercar · Arrastra para mover</div>
|
||||
</div>
|
||||
@@ -98,8 +215,29 @@ export function ProductGallery({ images, name }: { images: string[]; name: strin
|
||||
<span className="inline-flex h-11 items-center gap-2 rounded-full bg-fuchsia-500 px-4 text-sm font-black"><Maximize2 className="size-4" />{Math.round(scale * 100)}%</span>
|
||||
</div>
|
||||
</div>
|
||||
</DialogContent>
|
||||
</Dialog>
|
||||
</DialogContent>
|
||||
</Dialog>
|
||||
|
||||
{hoverZoom?.showPanel && (
|
||||
<div
|
||||
aria-hidden
|
||||
data-testid="gallery-zoom-panel"
|
||||
className="pointer-events-none fixed z-30 overflow-hidden rounded-[2rem] border border-slate-200 bg-white shadow-2xl"
|
||||
style={{
|
||||
left: hoverZoom.panelLeft,
|
||||
top: hoverZoom.panelTop,
|
||||
width: hoverZoom.panelSize,
|
||||
height: hoverZoom.panelSize,
|
||||
// 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",
|
||||
backgroundSize: `${HOVER_ZOOM_SCALE * 100}%`,
|
||||
backgroundPosition: `${hoverZoom.bgX}% ${hoverZoom.bgY}%`,
|
||||
}}
|
||||
/>
|
||||
)}
|
||||
</div>
|
||||
</div>
|
||||
);
|
||||
}
|
||||
@@ -1,6 +1,7 @@
|
||||
"use client";
|
||||
|
||||
import { CheckCircle2, ShoppingBag, TriangleAlert, Truck, XCircle } from "lucide-react";
|
||||
import { CheckCircle2, RotateCcw, ShoppingBag, TriangleAlert, Truck, XCircle } from "lucide-react";
|
||||
import Link from "next/link";
|
||||
import { useState } from "react";
|
||||
import { Button } from "@/components/ui/button";
|
||||
import type { Product } from "@/lib/headless/types";
|
||||
@@ -40,11 +41,14 @@ export function ProductPurchase({ product }: { product: Product }) {
|
||||
<strong className="text-3xl font-black">{formatMoney(product.price, product.currency)}</strong>
|
||||
{product.compareAtPrice && <del className="text-slate-400">{formatMoney(product.compareAtPrice, product.currency)}</del>}
|
||||
</div>
|
||||
{/* oculto en mobile: MobileAddToCart (barra fija abajo, lg:hidden) ya cubre esa misma accion ahi -- sin el hidden aqui, ambos se mostraban a la vez. */}
|
||||
{/* Un solo botón, visible también en mobile -- sin barra flotante
|
||||
aparte (se probó, pero terminaba tapando contenido de abajo al
|
||||
scrollear, ej. "También te puede interesar"; el usuario pidió
|
||||
sacarla y dejar el botón normal como en la mayoría de tiendas). */}
|
||||
<Button
|
||||
variant="primary"
|
||||
size="lg"
|
||||
className="hidden w-full lg:inline-flex"
|
||||
className="w-full"
|
||||
disabled={outOfStock}
|
||||
onClick={() => addItem(product, variantId)}
|
||||
>
|
||||
@@ -73,6 +77,21 @@ export function ProductPurchase({ product }: { product: Product }) {
|
||||
Envíos nacionales
|
||||
</span>
|
||||
</div>
|
||||
{/* Franja de confianza real (estilo Walmart) -- enlaza a las
|
||||
páginas reales de /envios y /cambios en vez de repetir texto
|
||||
suelto, con los datos reales que confirmó el usuario (5 días
|
||||
hábiles, envío de vuelta a cargo de ARI Shopping). Nada
|
||||
inventado: mismas políticas ya publicadas en el footer. */}
|
||||
<div className="mt-3 grid gap-2 text-sm">
|
||||
<Link href="/envios" className="flex items-center gap-2 font-bold text-slate-600 hover:text-ari-marino hover:underline">
|
||||
<Truck className="size-4 text-cyan-600" />
|
||||
Envíos calculados en el checkout · ver detalles
|
||||
</Link>
|
||||
<Link href="/cambios" className="flex items-center gap-2 font-bold text-slate-600 hover:text-ari-marino hover:underline">
|
||||
<RotateCcw className="size-4 text-ari-marino" />
|
||||
Cambios en 5 días hábiles, envío de vuelta gratis
|
||||
</Link>
|
||||
</div>
|
||||
</div>
|
||||
);
|
||||
}
|
||||
@@ -87,6 +87,18 @@ export function PredictiveSearch() {
|
||||
Buscando…
|
||||
</div>
|
||||
)}
|
||||
{/* Encontrado probando en vivo con navegador real: sin este bloque,
|
||||
una busqueda de 2+ caracteres sin coincidencias no mostraba nada
|
||||
(ni "buscando" ni un mensaje) -- el usuario se quedaba sin ningun
|
||||
feedback de que su busqueda de verdad corrio. Se ofrece igual el
|
||||
boton de buscar en todo el catalogo porque esa busqueda completa
|
||||
puede usar criterios mas amplios que el autocompletado. */}
|
||||
{!loading && normalized.length >= 2 && matches.length === 0 && (
|
||||
<div className="absolute left-0 right-0 top-[calc(100%+10px)] z-50 rounded-2xl border border-slate-200 bg-white p-2 text-center shadow-2xl">
|
||||
<p className="p-3 text-sm text-slate-500">No encontramos resultados para “{query.trim()}”.</p>
|
||||
<button type="submit" className="w-full rounded-xl bg-ari-marino px-4 py-3 text-sm font-black text-white hover:brightness-110">Buscar “{query.trim()}” en todo el catálogo</button>
|
||||
</div>
|
||||
)}
|
||||
{matches.length > 0 && (
|
||||
<div className="absolute left-0 right-0 top-[calc(100%+10px)] z-50 overflow-hidden rounded-2xl border border-slate-200 bg-white p-2 shadow-2xl">
|
||||
{matches.map((product) => (
|
||||
|
||||
@@ -25,8 +25,14 @@ spec:
|
||||
runAsUser: 1001
|
||||
containers:
|
||||
- name: app
|
||||
image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.238
|
||||
image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.273
|
||||
env:
|
||||
# La actualiza el mismo paso del CD (.gitea/workflows/build.yaml)
|
||||
# que bumpea el tag de la imagen -- el footer la lee
|
||||
# (process.env.APP_VERSION) para mostrar la versión real
|
||||
# desplegada en vez de un "v4.1" fijo.
|
||||
- name: APP_VERSION
|
||||
value: v1.0.273
|
||||
- name: HEADLESS_PROVIDER
|
||||
value: medusa
|
||||
- name: MEDUSA_BACKEND_URL
|
||||
|
||||
@@ -111,14 +111,26 @@ async function medusaFetch<T>(
|
||||
|
||||
const publishableKey = process.env.MEDUSA_PUBLISHABLE_KEY;
|
||||
|
||||
// Sin next.revalidate/tags a proposito (reportado por el usuario
|
||||
// 2026-09-20: imagenes editadas a mano en Medusa no se reflejaban en
|
||||
// el storefront, pero por CSV si). Causa real: frontend-deploy corre 2
|
||||
// replicas sin cacheHandler compartido -- cada pod tenia su propio
|
||||
// cache en memoria de este fetch (next.revalidate:60), y el POST de
|
||||
// /api/revalidate (revalidateTag) del subscriber de Medusa solo le
|
||||
// llegaba a UNO de los dos pods via el Service, dejando al otro
|
||||
// sirviendo la respuesta vieja hasta que el TTL expirara solo. Una
|
||||
// edicion puntual + revisar al toque tenia ~50% de chance de pegarle
|
||||
// al pod stale; una importacion CSV (minutos de duracion) le daba
|
||||
// tiempo de sobra al TTL para expirar en ambos pods, disimulando el
|
||||
// mismo bug. Las paginas que consumen esto ya son force-dynamic /
|
||||
// dinamicas por diseño (ver comentario en app/page.tsx y
|
||||
// app/product/[slug]/page.tsx) -- este next.revalidate competia contra
|
||||
// esa intencion en vez de cumplirla. Sin este bloque, Next 15 no
|
||||
// cachea el fetch por default dentro de una ruta dinamica.
|
||||
const response = await fetch(url, {
|
||||
headers: publishableKey
|
||||
? { "x-publishable-api-key": publishableKey }
|
||||
: undefined,
|
||||
next: {
|
||||
revalidate: 60,
|
||||
tags: ["medusa-products"],
|
||||
},
|
||||
});
|
||||
|
||||
if (!response.ok) {
|
||||
|
||||
Reference in new issue
Block a user