Fix: ranking usa fecha real de cierre (closed_at) en vez de "ahora" #1

marsalva · 2026-09-27T21:24:48Z

marsalva commented

2026-09-27 21:24:48 +00:00

Problema: el ranking calculaba "Cierre Rápido" (30/100 puntos) usando la hora actual en vez de la fecha real de cierre del servicio, por lo que ese apartado salía siempre en 0 para cualquier servicio creado hace más de 14 dias, aunque se hubiera cerrado rápido.

Cambios:

Nueva columna scraped_services.closed_at (migración segura, IF NOT EXISTS).
- El endpoint que cambia el estado de un servicio ahora guarda closed_at al pasar a estado final, y lo limpia si se reabre.
- calculateScore() usa closed_at real en vez de "ahora"; los servicios sin closed_at (cerrados antes de este fix) se excluyen de la media en vez de falsear el cálculo.
  Aviso: los servicios cerrados antes de fusionar esto no tienen closed_at, así que "Cierre Rápido" seguirá en 0 hasta que se cierren servicios nuevos con el fix ya activo.

Problema: el ranking calculaba "Cierre Rápido" (30/100 puntos) usando la hora actual en vez de la fecha real de cierre del servicio, por lo que ese apartado salía siempre en 0 para cualquier servicio creado hace más de 14 dias, aunque se hubiera cerrado rápido. Cambios: - Nueva columna scraped_services.closed_at (migración segura, IF NOT EXISTS). - - El endpoint que cambia el estado de un servicio ahora guarda closed_at al pasar a estado final, y lo limpia si se reabre. - - calculateScore() usa closed_at real en vez de "ahora"; los servicios sin closed_at (cerrados antes de este fix) se excluyen de la media en vez de falsear el cálculo. Aviso: los servicios cerrados antes de fusionar esto no tienen closed_at, así que "Cierre Rápido" seguirá en 0 hasta que se cierren servicios nuevos con el fix ya activo.

marsalva added 1 commit 2026-09-27 21:24:48 +00:00

Fix: ranking usa fecha real de cierre (closed_at) en vez de "ahora" b07868a207

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin fix/ranking-closed-at:fix/ranking-closed-at

git checkout fix/ranking-closed-at

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: marsalva/api#1