Remove duplicate ranking_max filter in get_teams #21

New Issue

cal · 2026-02-20T06:53:01Z

cal commented

2026-02-20 06:53:01 +00:00

`app/routers_v2/teams.py:136-143`. `ranking_max` filter applied twice. Second is a no-op — likely copy-paste error where `ranking_min` was intended.

Priority: low

\`app/routers_v2/teams.py:136-143\`. \`ranking_max\` filter applied twice. Second is a no-op — likely copy-paste error where \`ranking_min\` was intended. **Priority**: low

cal added the

bug

label 2026-02-20 06:53:01 +00:00

cal added the

ai-working

label 2026-03-03 22:00:59 +00:00

cal referenced this issue from a commit

2026-03-03 22:01:56 +00:00

fix: remove duplicate ranking_max filter in get_teams (#21)

cal referenced a pull request that will close this issue

2026-03-03 22:02:04 +00:00

fix: remove duplicate ranking_max filter in get_teams (#21) #42

cal removed the

ai-working

label 2026-03-03 22:02:11 +00:00

cal commented

2026-03-03 22:02:13 +00:00

Fixed in PR #42: #42

Removed the duplicate if ranking_max is not None block (lines 141-143) that was a copy-paste no-op of the filter already applied 3 lines earlier.

Fixed in PR #42: https://git.manticorum.com/cal/paper-dynasty-database/pulls/42 Removed the duplicate `if ranking_max is not None` block (lines 141-143) that was a copy-paste no-op of the filter already applied 3 lines earlier.

cal added the

ai-pr-opened

label 2026-03-03 22:02:16 +00:00

cal closed this issue

2026-03-05 03:17:51 +00:00

cal referenced this issue from a commit

2026-03-05 03:17:53 +00:00

Merge pull request 'fix: remove duplicate ranking_max filter in get_teams (#21)' (#42) from ai/paper-dynasty-database#21 into next-release

cal referenced this issue

2026-03-09 14:12:16 +00:00

Release: Scouting API, roster refactor, and bug fixes #61

Sign in to join this conversation.