Skip to content

fix: correct backup storage usage unit label#3016

Merged
ChiragAgg5k merged 1 commit intoappwrite:mainfrom
abhay-dev2901:fix-backup-storage-unit-label
May 3, 2026
Merged

fix: correct backup storage usage unit label#3016
ChiragAgg5k merged 1 commit intoappwrite:mainfrom
abhay-dev2901:fix-backup-storage-unit-label

Conversation

@abhay-dev2901
Copy link
Copy Markdown
Contributor

@abhay-dev2901 abhay-dev2901 commented May 2, 2026

What does this PR do?

Fixes an incorrect unit label in the organization usage breakdown.

Backup storage was being converted to GB, but the displayed label used MB. This updates the label to GB so the value and unit match.

Test Plan

Verified the code path manually by checking the organization usage breakdown calculation.

The backup storage value now uses bytesToSize(..., 'GB') and displays the GB unit consistently.

Related PRs and Issues

N/A

Have you read the Contributing Guidelines on issues (https://github.com/appwrite/appwrite/blob/master/CONTRIBUTING.md)?

Yes.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 2, 2026

Greptile Summary

This PR fixes a one-character typo in the backup storage tooltip label — the unit was incorrectly shown as MB while bytesToSize was converting to GB. The fix aligns it with all other storage tooltip labels in the same block (file storage, deployments storage, builds storage), which all already used GB correctly.

Note: The PR description describes adding unit tests for a variable detection helper and makes no mention of this tooltip fix. The description appears to have been copied from a different PR.

Confidence Score: 5/5

Safe to merge — the change is a trivial one-character label correction with no functional risk.

Single-line fix that matches the established pattern for all other storage tooltips in the same file. No logic changes, no regressions possible.

No files require special attention.

Important Files Changed

Filename Overview
src/routes/(console)/organization-[organization]/usage/[[invoice]]/+page.svelte Corrects the tooltip unit label for backup storage from 'MB' to 'GB', consistent with all other storage tooltip labels and the 'GB' argument passed to bytesToSize.

Reviews (1): Last reviewed commit: "fix: correct backup storage usage unit l..." | Re-trigger Greptile

@ChiragAgg5k ChiragAgg5k merged commit 59e7ff6 into appwrite:main May 3, 2026
1 check passed
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.

2 participants