tudor/school_compare

fix(api): legacy name-column fallback when marts predate the GIAS code migration #25

Merged

tudor merged 2 commits from fix/gias-legacy-fallback into main

2026-07-09 19:13:55 +00:00

Author	SHA1	Message	Date
TudorandClaude Fable 5	74ca76d150	fix(api): match missing-column fallbacks on the DBAPI error, not the statement PR Checks / Frontend Typecheck + Tests (pull_request) Successful in 9m41s Details PR Checks / Backend Smoke (pull_request) Successful in 6s Details PR Checks / Build Backend (no push) (pull_request) Successful in 20s Details PR Checks / Build Frontend (no push) (pull_request) Successful in 53s Details PR Checks / Build Pipeline (no push) (pull_request) Successful in 10s Details PR Checks / AI Code Review (Claude) (pull_request) Successful in 2m41s Details str(ProgrammingError) embeds the full SQL, which contains every column name — the substring check matched any error and could take the wrong retry branch. Parse the missing column from exc.orig instead. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>	2026-07-09 19:29:58 +01:00
TudorandClaude Fable 5	4b75152ee0	fix(api): fall back to legacy name-column query when marts predate code migration PR Checks / Frontend Typecheck + Tests (pull_request) Successful in 9m41s Details PR Checks / Backend Smoke (pull_request) Successful in 6s Details PR Checks / Build Backend (no push) (pull_request) Successful in 19s Details PR Checks / Build Frontend (no push) (pull_request) Successful in 47s Details PR Checks / Build Pipeline (no push) (pull_request) Successful in 10s Details PR Checks / AI Code Review (Claude) (pull_request) Failing after 3m8s Details Closes the deploy window flagged by CI review — the backend now works against both the old (name) and new (code) mart schemas. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>	2026-07-09 14:48:06 +01:00