fix(console): correct usage chart labels from functions/sites to deployments#2986
fix(console): correct usage chart labels from functions/sites to deployments#2986singhvishalkr wants to merge 4 commits into
Conversation
Greptile Summary
Confidence Score: 5/5Safe to merge based on the reviewed label and metric-source updates. The change is narrow, limited to usage-page metric loading and display text, with no reported functional regressions.
What T-Rex did
Reviews (3): Last reviewed commit: "fix(console): load aggregate deployment ..." | Re-trigger Greptile |
Retain usage Deployments wording; upstream stubs replaced with ours during conflict.
|
Merged current |
|
Updated this in The aggregate Functions and Sites usage loaders now fetch deployment usage through the SDK instead of returning
I also verified the pinned |
Summary
Fixes #2224
The usage chart on both the Functions and Sites aggregate usage pages displays incorrect labels. The data source is
deploymentsTotal/deployments, but the chart metadata says "Total functions" / "Total sites" respectively.Changes
legendfrom "Functions" to "Deployments" andtitlefrom "Total functions" to "Total deployments"legendfrom "Sites" to "Deployments" andtitlefrom "Total sites" to "Total deployments"Both pages load deployment metrics from the API (
data.deploymentsTotal,data.deployments), so the labels should reflect that.Files changed
src/routes/(console)/project-[region]-[project]/functions/usage/[[period]]/+page.sveltesrc/routes/(console)/project-[region]-[project]/sites/usage/[[period]]/+page.svelteTest plan