Skip to content

fix(schemas): remove Zombies' overall fastest win and its leaderboard - #1012

Open
ugcodrr wants to merge 4 commits into
mainfrom
fix/zombies-overall-fastest-win
Open

ugcodrr wants to merge 4 commits into
mainfrom
fix/zombies-overall-fastest-win

Conversation

@ugcodrr

@ugcodrr ugcodrr commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Description: Zombies' overall Fastest Win showed Hypixel's fastest_time_30_zombies, which isn't the fastest of a player's map wins, so the stat and its leaderboard were wrong.

  1. Zombies Overall Wins loses its Fastest Win column, and /rankings loses that row.
image

Merge order for the perf PRs is in #1002.

Testing: /leaderboard arcade, then pick Zombies Overall Wins. There's no Fastest Win column.

Zombies as a whole has no fastest win, only each map's. Zombies.overall
was a ZombiesMapDifficulty built without a map, so it stored Hypixel's
fastest_time_30_zombies, which isn't the fastest of the maps' wins, and
ranked it on its own leaderboard and as the extra column of the Overall
Wins leaderboard.

Zombies.overall is now a ZombiesOverall with the same fields minus
fastestWin, so that leaderboard goes away and Overall Wins has no extra
column. Each map's overall fastest win, the fastest of its difficulties,
is unchanged, and so is Alien Arcadium's.
@vercel

vercel Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
statsify Ignored Ignored Oct 22, 2026 4:29am UTC

Removed comment about fastest win for zombies across maps.
@vercel

vercel Bot commented Nov 3, 2026

Copy link
Copy Markdown

Deployment failed for project statsify with the following error:

Too many requests - try again in 5 minutes (more than 60, code: "api-deployments-flood").

Learn More: https://vercel.com/docs/limits#rate-limits

This branch had an error being deployed

1 failed (outdated) deployment
Preview — e4cc892b Deployed Oct 21, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant