Compare commits
No commits in common. "9fddb71183fb0f17dfa06b42be1cc5b039f5841c" and "87b82ca5f1fbdd67c4ce195f7b98fd6a44eec912" have entirely different histories.
9fddb71183
...
87b82ca5f1
@ -86,7 +86,3 @@ 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
|
||||
|
||||
@ -1,24 +0,0 @@
|
||||
---
|
||||
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`.
|
||||
Loading…
Reference in New Issue
Block a user