fix: correct usage labels from 'Total functions/sites' to 'Total deployments'#2581
fix: correct usage labels from 'Total functions/sites' to 'Total deployments'#2581woustachemax wants to merge 1 commit intoappwrite:mainfrom
Conversation
…al functions/sites'
Console (appwrite/console)Project ID: Sites (1)
Tip Environment variables can be scoped per function or shared across your project |
WalkthroughThis PR updates display labels in two usage tracking pages. The Functions usage page changes a legend label from "Total functions" to "Total Deployments", and the Sites usage page changes a title from "Total sites" to "Total site Deployments". Both changes are text updates only, with no modifications to underlying logic or data handling. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Pre-merge checks and finishing touches✅ Passed checks (5 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/routes/(console)/project-[region]-[project]/sites/usage/[[period]]/+page.svelte (1)
16-16: Label correction looks good, minor terminology inconsistency.The change correctly reflects that deployments are being counted (confirmed by
data.deploymentsTotalon Line 7). However, there's a minor inconsistency with the Functions page, which uses "Total Deployments" while this uses "Total site Deployments". Since thelegendfield already provides context ('Sites'), consider using "Total Deployments" for consistency across both pages.Optional diff for consistency:
- title: 'Total site Deployments' + title: 'Total Deployments'
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/routes/(console)/project-[region]-[project]/functions/usage/[[period]]/+page.svelte(1 hunks)src/routes/(console)/project-[region]-[project]/sites/usage/[[period]]/+page.svelte(1 hunks)
🔇 Additional comments (1)
src/routes/(console)/project-[region]-[project]/functions/usage/[[period]]/+page.svelte (1)
16-16: LGTM! Label accurately reflects the deployment metric.The change correctly updates the title to "Total Deployments", which aligns with the data being displayed (
data.deploymentsTotalon Line 7). Thelegendfield ('Functions') already provides the necessary context.

Fixes #2224
The usage pages for Functions and Sites were showing labels that said "Total functions" and "Total sites", but the actual metric being displayed was the total number of deployments across all functions/sites.
Changed both labels to "Total deployments" to accurately reflect what's being counted.
This is a simple label fix - I know it's not a complex PR, just correcting the misleading text.
Changes
Summary by CodeRabbit