Skip to content

fix(character): rankings tab shows curated bosses only + expansion dropdown - #191

Merged
VortexUK merged 1 commit into
mainfrom
fix/rankings-tab-curated-only
Jul 27, 2026
Merged

fix(character): rankings tab shows curated bosses only + expansion dropdown#191
VortexUK merged 1 commit into
mainfrom
fix/rankings-tab-curated-only

Conversation

@VortexUK

Copy link
Copy Markdown
Owner

Summary

Follow-up to #190 (user feedback: "it's showing every conceivable boss").

  • The tab is now gated to the curated zones.db universe — the same (zone → bosses) set the rankings page dropdowns show. Heuristic-matched kills (any capitalised mob title) are dropped entirely: no boss rows, no percentile-pool or All Stars pollution, and a character with only uncurated parses keeps the tab hidden.
  • Zone sections carry their zones.db expansion tag; the response lists the expansions the character has ranked kills in (newest first — currently that can only be EoF/RoK, matching the curated raid section). The tab adds an expansion dropdown next to the Damage/Healing toggle, defaulting to the newest, filtering client-side with no refetch.

Test plan

  • Backend: +2 tests (a 99,999-DPS kill in an uncurated zone must never surface; an all-uncurated character gets zones: []), fixture now patches the curated-tree seam. 8/8 green, full suite 1638.
  • Frontend: +1 dropdown-filtering test (two expansions, newest default, switch swaps sections). 257 green, tsc + build clean.
  • Live smoke against seeded local parses: Freethinker Hideout tagged EoF, only its 4 curated bosses returned.

🤖 Generated with Claude Code

…opdown

The tab was surfacing every heuristic-matched named (any capitalised mob
title) the character ever logged. Now gated to the curated zones.db
universe - the exact set the rankings page dropdowns show. Uncurated
kills are dropped entirely: no boss rows, no percentile-pool or All
Stars pollution, and an all-uncurated character keeps the tab hidden.

Zone sections carry their zones.db expansion tag and the response lists
the expansions the character has ranked kills in (newest first); the tab
adds an expansion dropdown next to the metric toggle, defaulting to the
newest, filtering client-side.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@VortexUK
VortexUK merged commit 2f879cd into main Jul 27, 2026
6 checks passed
@VortexUK
VortexUK deleted the fix/rankings-tab-curated-only branch July 27, 2026 18:38
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.

1 participant