Skip to content

Cleanup leaderboard ordering and N/a scores#2337

Open
Didayolo wants to merge 1 commit intodevelopfrom
na-scores
Open

Cleanup leaderboard ordering and N/a scores#2337
Didayolo wants to merge 1 commit intodevelopfrom
na-scores

Conversation

@Didayolo
Copy link
Copy Markdown
Member

@Didayolo Didayolo commented Apr 15, 2026

Description

  • Fix logic if (score) that returns N/a when score is 0
  • Add helpers in the front-end to sort values instead of displayed strings
  • Makes the back-end robust and use nulls_last=True filter to have N/a at the bottom of the sorting even if the rule is "descending".

Issue the PR resolves

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

@Didayolo
Copy link
Copy Markdown
Member Author

CircleCI: need to merge #2338 and rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

N/a are displayed on top of the leaderboard

1 participant