Skip to content

feat(leaderboards): add value search - #1053

Draft
ugcodrr wants to merge 10 commits into
mainfrom
feat/leaderboard-value-search-button
Draft

ugcodrr wants to merge 10 commits into
mainfrom
feat/leaderboard-value-search-button

Conversation

@ugcodrr

@ugcodrr ugcodrr commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Description: There's no way to jump to where a value would rank on a leaderboard, like 500✫ on BedWars level or 1:23.45 on a parkour time.

  1. Level leaderboards take a level (a prestige on Pit), times also take units like 5d 3h, and a value past the end of a leaderboard shows an error.

Testing: /leaderboard bedwars leaderboard:Level, then the new button with 500✫, jumps to the page where 500✫ starts.

Merge after #1002, #1003, #926 and #1052. #926 and #1003 each conflict with it on a line or two.

Supersedes #894

/leaderboard bedwars Level, before and after

@vercel

vercel Bot commented Oct 30, 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 Nov 5, 2026 6:34am UTC

const search = input.replaceAll(",", "").trim().toLowerCase();

// Times are searched in seconds, like 1:23.45 or 83.45, or in units, like 5d 3h, but stored in milliseconds
if (formatter === formatTime || formatter === formatRaceTime) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This probably shouldn't be hard coded for stats. It would be better if when passing a formatter, you also pass a parser to read the value back for searching.

@jacobk999
jacobk999 marked this pull request as draft November 1, 2026 19:53
@vercel

vercel Bot commented Nov 3, 2026

Copy link
Copy Markdown

Deployment failed for project statsify with the following error:

Resource is limited - try again in 1 day (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/statsify-net?upgradeToPro=build-rate-limit

This branch has not been deployed

No deployments
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.

2 participants