Skip to content

fix: point backup policy upgrade link to enterprise contact page#3093

Merged
HarshMN2345 merged 1 commit into
mainfrom
fix-backup-policy-upgrade-link
Jun 24, 2026
Merged

fix: point backup policy upgrade link to enterprise contact page#3093
HarshMN2345 merged 1 commit into
mainfrom
fix-backup-policy-upgrade-link

Conversation

@HarshMN2345

Copy link
Copy Markdown
Member

Summary

  • Pro plan users hitting the 1-policy backup limit were sent to the change-plan page, which only shows Free/Pro options — leaving them confused since they were already on Pro
  • The "Upgrade your plan" links in createPolicy.svelte now point to https://appwrite.io/contact-us/enterprise and open in a new tab
  • Text updated to "Upgrade to Enterprise" to clearly communicate what the upgrade path is
  • Removed now-unused imports: goto, getChangePlanUrl, showCreatePolicy

Test plan

  • On a Pro plan org, open the backup policies section for a database
  • Verify the "Upgrade to Enterprise" link opens https://appwrite.io/contact-us/enterprise in a new tab
  • Verify the same link appears in the database creation wizard backup step

Pro plan users hitting the 1-policy limit were sent to the change-plan
page which only shows Free/Pro options, leaving them confused since they
were already on Pro. Now links directly to the enterprise contact page.
@greptile-apps

greptile-apps Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Pro plan users hitting the 1-policy backup limit were previously navigated to the change-plan page, which only shows Free/Pro options and left them stuck. This PR replaces both internal navigation handlers with external links to https://appwrite.io/contact-us/enterprise and updates the button text to "Upgrade to Enterprise".

  • Both occurrences of the upgrade prompt (isFromBackupsTab branch and the InputSwitch description) now open the enterprise contact page in a new tab instead of routing within the app.
  • Unused imports (goto, getChangePlanUrl, showCreatePolicy) are cleaned up correctly.

Confidence Score: 5/5

Safe to merge — minimal, well-scoped change that only affects the upgrade CTA links shown to Pro plan users in the backup policy UI.

The change is purely a UI link fix: two click handlers that navigated internally are replaced with external anchor targets, and three imports that became dead code are removed. No logic, state, or data flow is altered beyond what the PR intends.

No files require special attention.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/databases/database-[database]/backups/createPolicy.svelte Replaces two on:click navigation handlers (which sent Pro users to the change-plan page) with direct external links to the enterprise contact page; removes now-unused goto, getChangePlanUrl, and showCreatePolicy imports.

Reviews (1): Last reviewed commit: "fix: point backup policy upgrade link to..." | Re-trigger Greptile

@HarshMN2345 HarshMN2345 merged commit c1d66e2 into main Jun 24, 2026
3 of 4 checks passed
@HarshMN2345 HarshMN2345 deleted the fix-backup-policy-upgrade-link branch June 24, 2026 14:34
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