Skip to content

fix: use currentPlan for backup banner availability#2860

Merged
HarshMN2345 merged 1 commit intomainfrom
fix-backup-banner-cta
Feb 16, 2026
Merged

fix: use currentPlan for backup banner availability#2860
HarshMN2345 merged 1 commit intomainfrom
fix-backup-banner-cta

Conversation

@HarshMN2345
Copy link
Member

@HarshMN2345 HarshMN2345 commented Feb 16, 2026

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

  • Refactor
    • Updated how backup availability information is sourced in the backup database alert component.

@appwrite
Copy link

appwrite bot commented Feb 16, 2026

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

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

Tip

Appwrite has crossed the 50K GitHub stars milestone with hundreds of active contributors

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 16, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

The pull request modifies src/lib/components/backupDatabaseAlert.svelte by adding an import for the currentPlan store from src/lib/stores/organization and updating the source of the backupsEnabled check. The areBackupsAvailable value now derives from $currentPlan?.backupsEnabled instead of $organization?.billingPlanDetails.backupsEnabled. The component's control flow and UI rendering logic remain unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: use currentPlan for backup banner availability' directly and accurately describes the main change: replacing the source of backup availability from organization.billingPlanDetails to currentPlan store.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-backup-banner-cta

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.

@HarshMN2345 HarshMN2345 merged commit 2cf0cf8 into main Feb 16, 2026
4 of 6 checks passed
@HarshMN2345 HarshMN2345 deleted the fix-backup-banner-cta branch February 16, 2026 08:22
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