fix(rankings): expose selected metric under stable value key
Build and Push Docker Images / Build Backend (FastAPI) (push) Successful in 22s
Build and Push Docker Images / Build Frontend (Next.js) (push) Successful in 55s
Build and Push Docker Images / Build Pipeline (Meltano + dbt + Airflow) (push) Successful in 12s
Build and Push Docker Images / Trigger Portainer Update (push) Successful in 0s
Build and Push Docker Images / Build Backend (FastAPI) (push) Successful in 22s
Build and Push Docker Images / Build Frontend (Next.js) (push) Successful in 55s
Build and Push Docker Images / Build Pipeline (Meltano + dbt + Airflow) (push) Successful in 12s
Build and Push Docker Images / Trigger Portainer Update (push) Successful in 0s
The /api/rankings endpoint returned each row keyed by the metric's column name (e.g. rwm_high_pct) but never under a generic `value` field. The frontend RankingItem type and RankingsView both read ranking.value, so every row rendered "—" for every metric — the default rwm_expected_pct included. Add `df["value"] = df[metric]` before JSON serialisation so the frontend gets the value it has always expected. The raw metric column is still in the row for any caller that wants it explicitly. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
+6
-1
@@ -818,7 +818,12 @@ async def get_rankings(
|
|||||||
|
|
||||||
# Return only relevant fields for rankings
|
# Return only relevant fields for rankings
|
||||||
available_cols = [c for c in RANKING_COLUMNS if c in df.columns]
|
available_cols = [c for c in RANKING_COLUMNS if c in df.columns]
|
||||||
df = df[available_cols]
|
df = df[available_cols].copy()
|
||||||
|
|
||||||
|
# Surface the requested metric under a stable `value` key so the
|
||||||
|
# frontend doesn't need to know each metric's column name. The raw
|
||||||
|
# metric column is also kept in the row for callers that want it.
|
||||||
|
df["value"] = df[metric]
|
||||||
|
|
||||||
return {
|
return {
|
||||||
"metric": metric,
|
"metric": metric,
|
||||||
|
|||||||
Reference in New Issue
Block a user