store: Fix: SBA API transaction move_id is a string not i64

This commit is contained in:
Cal Corum 2026-02-28 10:52:56 -06:00
parent 84ec45a077
commit af1e6700a9

View File

@ -0,0 +1,26 @@
---
id: d5b6584e-ae9d-4462-aeb0-eac103156dfc
type: fix
title: "Fix: SBA API transaction move_id is a string not i64"
tags: [sba-scout, rust, api, serde, fix, transactions]
importance: 0.6
confidence: 0.8
created: "2026-02-28T16:52:56.737424+00:00"
updated: "2026-02-28T16:52:56.737424+00:00"
---
# Transaction move_id as String
**Project:** sba-scout (Rust) | **Commit:** `60b397b`
## Problem
The SBA league API returns transaction `moveid` as a formatted string like `"Season-013-Week-11-1772073335"`, not an integer. Deserializing as `Option<i64>` panics/errors.
## Fix
Changed `move_id` from `Option<i64>` to `Option<String>` in `api/types.rs` `TransactionData` struct.
## Pattern
Don't assume numeric-looking field names in an API are actually integers. Inspect real API responses before choosing Rust types. Transaction/move identifiers often encode semantic information as composite strings.