Skip to content

Conversation

@woustachemax
Copy link

@woustachemax woustachemax commented Nov 11, 2025

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

  • Updated Functions usage page label
  • Updated Sites usage page label (mentioned in issue comments)

Summary by CodeRabbit

  • Style
    • Updated Functions usage view label to display "Total Deployments"
    • Updated Sites usage view label to display "Total site Deployments"

@appwrite
Copy link

appwrite bot commented Nov 11, 2025

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Failed Failed Authorize Preview URL QR Code

Tip

Environment variables can be scoped per function or shared across your project

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 11, 2025

Walkthrough

This 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

  • Verify the new labels accurately reflect what metrics are being displayed
  • Check that terminology is consistent with other deployment-related labels in the codebase

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change—correcting usage labels from 'Total functions/sites' to 'Total deployments' to reflect the actual metric displayed.
Linked Issues check ✅ Passed Changes directly address issue #2224 by correcting both the Functions and Sites usage labels from 'Total functions/sites' to 'Total deployments' to accurately reflect the displayed deployment metrics.
Out of Scope Changes check ✅ Passed All changes are scoped to correcting usage labels in the Functions and Sites pages as required by issue #2224, with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.deploymentsTotal on Line 7). However, there's a minor inconsistency with the Functions page, which uses "Total Deployments" while this uses "Total site Deployments". Since the legend field 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

📥 Commits

Reviewing files that changed from the base of the PR and between 73552e5 and b6bbd43.

📒 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.deploymentsTotal on Line 7). The legend field ('Functions') already provides the necessary context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 Bug Report: Functions usage typo

1 participant