feat: /teams/{id}/evolutions should sort by current_value desc #116

New Issue

cal · 2026-03-19T16:37:44Z

cal commented

2026-03-19 16:37:44 +00:00

Description

GET /api/v2/teams/{team_id}/evolutions returns results ordered by player_id (insertion order). This means /evo status in Discord shows a wall of T0/value-0 cards at the top instead of the most progressed players.

Current Behavior

Results ordered by player_id — unfiltered view starts with unranked zero-progress cards.

Expected

Default sort by current_tier DESC, current_value DESC so the most evolved players appear first. Optionally support a sort query param for other orderings.

Impact

Without this, /evo status is unusable without the tier:1 filter — users see 10 pages of zeros before reaching any interesting data.

Found during

Phase 1 smoke testing (2026-03-19)

## Description `GET /api/v2/teams/{team_id}/evolutions` returns results ordered by `player_id` (insertion order). This means `/evo status` in Discord shows a wall of T0/value-0 cards at the top instead of the most progressed players. ## Current Behavior Results ordered by player_id — unfiltered view starts with unranked zero-progress cards. ## Expected Default sort by `current_tier DESC, current_value DESC` so the most evolved players appear first. Optionally support a `sort` query param for other orderings. ## Impact Without this, `/evo status` is unusable without the `tier:1` filter — users see 10 pages of zeros before reaching any interesting data. ## Found during Phase 1 smoke testing (2026-03-19)

cal referenced this issue from a commit

2026-03-19 18:14:29 +00:00

fix: sort /teams/{id}/evolutions by current_tier desc, current_value desc (#116)

cal referenced a pull request that will close this issue

2026-03-19 18:14:36 +00:00

fix: sort /teams/{id}/evolutions by current_tier desc, current_value desc (#116) #120

cal commented

2026-03-19 18:14:43 +00:00

PR opened: #120

The fix is a one-line change in list_team_evolutions — replaced .order_by(EvolutionCardState.player_id) with .order_by(current_tier DESC, current_value DESC). Most-progressed cards will now appear at the top of /evo status without needing any filter. All 117 tests pass.

PR opened: https://git.manticorum.com/cal/paper-dynasty-database/pulls/120 The fix is a one-line change in `list_team_evolutions` — replaced `.order_by(EvolutionCardState.player_id)` with `.order_by(current_tier DESC, current_value DESC)`. Most-progressed cards will now appear at the top of `/evo status` without needing any filter. All 117 tests pass.

cal referenced this issue from a commit

2026-03-19 18:24:22 +00:00

fix: sort /teams/{id}/evolutions by current_tier desc, current_value desc (#116)

cal closed this issue

2026-03-19 18:24:49 +00:00

cal referenced this issue from a commit

2026-03-19 18:24:50 +00:00

Merge pull request 'fix: sort /teams/{id}/evolutions by current_tier desc, current_value desc (#116)' (#120) from ai/paper-dynasty-database-116 into next-release

cal referenced this issue

2026-03-24 05:16:10 +00:00

ci: database CI catchup — local buildx cache + dev tag trigger #153

Sign in to join this conversation.