Revisión detallada del avance 2 — Santos Tirado y Cruz Sandoval

❌ C2 · Carrito que persiste en localStorage — 0/3

Qué pasa: el carrito agrega, suma/resta, quita y calcula el total, pero no guarda nada en localStorage. Al recargar la página el carrito se pierde.

Prueba en el navegador:

• Agregué 2 × Torta Cubana ($110) y 1 × Pierna Adobada ($95) → Total: $315.00 ✔️
• Object.keys(localStorage) después de agregar → [] (vacío).
• Recargué la página → «El carrito está vacío.» y Total: $0.00.

Por qué está mal: en js/carrito.js el carrito es solo una variable en memoria, que se borra cada vez que se recarga:

let carrito = [];          // línea 1: vive solo mientras la página está abierta
...
function agregarAlCarrito(nombre, precio) {
    ...
    renderizarCarrito();    // pinta, pero nunca llama a localStorage.setItem(...)
}

No hay ninguna llamada a localStorage.getItem ni localStorage.setItem en todo el repositorio (tampoco en servidor-ssr-stjg/.../static/js/carrito.js). La persistencia es justo lo que pide el paso 14 y el criterio dice «que persiste en localStorage», así que sin ella el criterio no se cumple.

Cómo debió quedar (mínimo):

let carrito = JSON.parse(localStorage.getItem("carrito") || "[]");
function guardar() { localStorage.setItem("carrito", JSON.stringify(carrito)); }
// llamar guardar() en agregar, cambiarCantidad y eliminar,
// y renderizarCarrito() en DOMContentLoaded para pintarlo al cargar.

❌ C4 · README con capturas y hash del avance 1 — 0/1

Las dos capturas sí existen y sí están insertadas como imagen, pero el criterio pide también el encabezado actualizado con el hash del avance 1, y eso falta:

• Falta el hash del avance 1 (motivo principal). La tabla «Historial de Avances» solo tiene descripciones y fechas. El commit que entregó en el avance 1 fue 6a6f295 y no aparece en ningún lugar del README.
• Grupo incorrecto: dice TDSM10-1; el grupo es ING10-1.
• Nombres de las capturas: usó docs/captura-ssr-stjg.png y docs/captura-csr-stjg.png; la instrucción pide exactamente docs/captura-ssr.png y docs/captura-csr.png. (Esto solo no le habría quitado el punto: las imágenes se ven.)
• Además, el README tiene pegada la plantilla de Django en cursiva (*<div ...> {% for producto in productos %} ...*), que GitHub muestra como texto roto.

❌ C5 · 3 commits en 3 días distintos — 0/1

Commits del avance 2 (después del commit del avance 1 6a6f295 y hasta el entregado):

6065cdf  13 oct 10:38
6c60ec9  13 oct 11:19
13b54bb  13 oct 19:09
08fe65c  15 oct 23:02   ← entregado

Son 4 commits, pero solo en 2 días distintos (13 y 15 oct). La regla dice que con menos de 3 días distintos el criterio vale 0, sin importar cuántos commits haya.

❌ C4 · README con capturas y hash del avance 1 — 0/1

• Falta insertar la captura CSR. El archivo docs/captura-csr.png sí existe en el repo, pero bajo el título «CSR (JavaScript): el div del catálogo llega vacío» no hay ninguna línea ![Código fuente CSR](docs/captura-csr.png). En GitHub solo se ve 1 de las 2 imágenes.
• Hash del avance 1 equivocado. El historial dice de78377, pero el commit que entregó en el avance 1 fue e7f96a1 (el último de los tres commits «Avance 1» del 10 oct: de78377 8:29 pm → 36903b9 8:40 pm → e7f96a1 8:42 pm). Anotó el primero, no el entregado.

¿Qué significa «anotó el primero, no el entregado»?

El 10 de octubre hizo tres commits seguidos, todos con el mensaje «Avance 1»:

En el formulario del avance 1 entregó e7f96a1, el último, y ese es el que se evaluó. Pero en el «Historial de avances» del README escribió de78377, el primero de los tres.

Por qué es un error y no solo un detalle: cada hash identifica una versión exacta del repositorio. de78377 es una versión en la que todavía no existía el README; la versión revisada y calificada es e7f96a1. Con el hash equivocado, quien lea el historial abre una versión distinta de la evaluada.

Por qué probablemente se confundió: escribió la tabla del README a las 8:40, justo después del commit del código, y anotó ese hash. Después hizo dos commits más y entregó el último. Es cierto que un archivo no puede contener el hash de su propio commit, así que en el avance 1 no podía anotar e7f96a1. Pero el avance 2 lo hizo el 16 de octubre, cuando e7f96a1 ya existía y ya lo había entregado: ahí debió corregirlo.

Cualquiera de las dos faltas basta para no cumplir el criterio: aunque se le perdonara el hash, la captura CSR sigue sin estar insertada en el README.

❌ C5 · 3 commits en 3 días distintos — 0/1

Entre el commit del avance 1 (e7f96a1, 10 oct) y el entregado hay un solo commit:

817ec7d  16 oct 23:30   ← entregado (todo el avance 2: 32 archivos, 474 líneas)

Todo el avance se subió en un solo commit, en un solo día.