feat: Leaderboard configurable - ordenamiento, tamaño, posición del jugador#14
Open
michumichifu wants to merge 1 commit intolino9999:masterfrom
Open
feat: Leaderboard configurable - ordenamiento, tamaño, posición del jugador#14michumichifu wants to merge 1 commit intolino9999:masterfrom
michumichifu wants to merge 1 commit intolino9999:masterfrom
Conversation
…ugador, barra XP - Ordenamiento del leaderboard configurable via config.yml (sort-by, tiebreaker) - Tamaño configurable (default 20, max 27, antes hardcodeado a 10) - Filtro min-level para excluir jugadores inactivos - Item "Tu Posición" con ranking personal eficiente via COUNT(*) - Barra de progreso XP visual en el lore - Opción para resetear total_levels al cambiar temporada - Corrige que el leaderboard ordenaba por total_levels acumulativo
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
1 participant
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.
Resumen
El leaderboard actualmente ordena por total_levels, un contador acumulativo que nunca se resetea al cambiar de temporada. Esto hace que los jugadores veteranos ocupen permanentemente las primeras posiciones sin importar su actividad en la temporada actual.
Este PR agrega configurabilidad completa al leaderboard y corrige el reset de temporada.
Cambios
Ordenamiento configurable (config.yml):
Posición del jugador: Nuevo item "Tu Posición" (RECOVERY_COMPASS, slot 31) que muestra la posición en el ranking aunque no esté en el top. Usa query eficiente con COUNT(*).
Barra de progreso XP: Barra visual en el lore de cada jugador del top.
Reset de temporada: resetSeason() ahora opcionalmente resetea total_levels a 0, para que el leaderboard refleje la temporada actual.
El problema (antes de este PR)
Retrocompatible