Compare commits

...

2 Commits

2 changed files with 28 additions and 0 deletions

View File

@ -86,3 +86,7 @@ Errors
## 12:09 - Fix: Dashboard didn't refresh after sync — re-mount on SyncComplete
- **Type:** fix
- **Tags:** sba-scout, rust, dashboard, ux, sync, tui
## 12:09 - Fix: Dashboard position coverage only counted pos_1, not all positions
- **Type:** fix
- **Tags:** sba-scout, rust, dashboard, positions, bug-fix

View File

@ -0,0 +1,24 @@
---
id: 6d3627c4-5aa1-47f0-a85f-34fe3ea685ac
type: fix
title: "Fix: Dashboard position coverage only counted pos_1, not all positions"
tags: [sba-scout, rust, dashboard, positions, bug-fix]
importance: 0.55
confidence: 0.8
created: "2026-02-28T18:09:34.355896+00:00"
updated: "2026-02-28T18:09:34.355896+00:00"
---
# Fix: Dashboard position coverage only counted pos_1, not all positions
## Problem
Position coverage calculation in the dashboard only iterated `player.pos_1` for each player, missing multi-position eligibility (e.g. a player with 2B/3B/SS eligibility was only counted in pos_1).
## Fix
Changed to use the existing `Player::positions()` helper method (defined in `db/models.rs`) which iterates all 8 position fields (`pos_1` through `pos_8`). Now a player with 2B/3B/SS eligibility is counted once in each eligible position bucket.
## Committed
Commit `a033a28`.