Agregar tabla de posiciones virtual por jugador en Torneo Nacional - #1
Merged
Conversation
- Nuevo endpoint GET /api/estadisticas-nacional/tabla-posiciones-virtual (protegido con verifyToken): calcula la tabla de posiciones usando los goles que el usuario pronosticó, considerando solo partidos ya jugados para los que entregó pronóstico. Mismo sistema de puntos y criterios automáticos de desempate que la tabla real. - Página EstadisticasNacional: agrega una sección "Mi Tabla de Posiciones Virtual" debajo de la tabla real y del historial de partidos, visible solo para el usuario autenticado y calculada con sus propios pronósticos.
✅ Deploy Preview for pollafutbol ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Este PR agrega una “tabla de posiciones virtual” para el Torneo Nacional, calculada por usuario autenticado en base a sus propios pronósticos (solo partidos ya jugados y con pronóstico entregado), y la muestra en la página de Estadísticas junto a la tabla real.
Changes:
- Backend: nuevo endpoint protegido
GET /api/estadisticas-nacional/tabla-posiciones-virtualque calcula estadísticas/puntos usandopronosticos.goles_local/goles_visitadel usuario. - Backend: agrega métricas de cobertura (
partidos_pronosticadosytotal_partidos_jugados) para contextualizar la tabla virtual. - Frontend: nueva sección “Mi Tabla de Posiciones Virtual” con estados de loading/error y mensajes cuando no hay sesión o no existen pronósticos jugados.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| server/routes/estadisticas_nacional.js | Agrega endpoint autenticado para calcular y retornar la tabla virtual y métricas de cobertura. |
| client/src/pages/EstadisticasNacional.jsx | Consume el nuevo endpoint y renderiza la tabla virtual con UI de estados y mensajes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+272
to
+275
| res.status(500).json({ | ||
| error: 'Error obteniendo tabla de posiciones virtual', | ||
| details: error.message | ||
| }); |
Comment on lines
+92
to
+96
| } catch (error) { | ||
| console.error('Error cargando tabla virtual:', error); | ||
| setErrorVirtual('No se pudo cargar tu tabla de posiciones virtual.'); | ||
| } finally { | ||
| setLoadingVirtual(false); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Qué hace
En la página Estadísticas del Torneo Nacional, cada jugador logueado ahora ve, además de la tabla de posiciones real, su propia tabla de posiciones virtual, calculada con los goles que él mismo pronosticó (solo para partidos ya jugados y con pronóstico entregado).
Cambios
server/routes/estadisticas_nacional.js): nuevo endpointGET /api/estadisticas-nacional/tabla-posiciones-virtual(protegido converifyToken). Calcula la tabla usandopronosticos.goles_local/goles_visitadel usuario autenticado en lugar de los goles reales, solo para partidos con resultado real ya cargado. Mismo sistema de puntos (3/1/0) y mismos 4 criterios automáticos de desempate que la tabla real (puntos, diferencia de gol, ganados, goles a favor). No aplica los criterios manuales 5-8 (son específicos de la tabla real del torneo).client/src/pages/EstadisticasNacional.jsx): nueva sección "🔮 Mi Tabla de Posiciones Virtual" debajo de la tabla real y el historial de partidos. Muestra mensajes claros si no hay sesión iniciada o si el usuario aún no tiene pronósticos de partidos ya jugados.Cómo probar
/estadisticas-nacional.🤖 Generated with Claude Code
Generated by Claude Code