store: FullCard migration: database-side schema + router changes

This commit is contained in:
Cal Corum 2026-02-25 16:26:54 -06:00
parent 2d42747388
commit 51ad67f8e6

View File

@ -0,0 +1,12 @@
---
id: 09bb95a5-2b2f-43e5-93a6-6b96b881461f
type: decision
title: "FullCard migration: database-side schema + router changes"
tags: [paper-dynasty, database, db_engine, battingcardratings, pitchingcardratings, migration, schema, routers_v2]
importance: 0.8
confidence: 0.8
created: "2026-02-25T22:26:54.133040+00:00"
updated: "2026-02-25T22:26:54.133040+00:00"
---
## Branch\nfeature/fullcard-migration in the database repo.\n\n## Files Updated\n\n### db_engine.py\nAdded 9 nullable TextField columns to BattingCardRatings and PitchingCardRatings:\n- col_one_2d6, col_one_results, col_one_d20\n- col_two_2d6, col_two_results, col_two_d20\n- col_three_2d6, col_three_results, col_three_d20\n\n### routers_v2/battingcardratings.py + pitchingcardratings.py\nAdded 9 Optional[str] = None fields to Pydantic models. PUT handler uses x.dict() so it persists new fields automatically — no handler code changes needed.\n\n### card_creation.py\nget_batter_card_data() and get_pitcher_card_data() check if ratings_vl.col_one_2d6 is not None and ratings_vr.col_one_2d6 is not None — if so, return stored columns directly (skips full on-the-fly rebuild for backwards compat).\n\n## Migration SQL\nALTER TABLE battingcardratings ADD COLUMN col_one_2d6 TEXT (and 8 more) is INTENTIONALLY NOT RUN — documented but blocked until reviewed.