Skip to content

fix(studio): extend downgrade error toast duration - #47780

Merged
awaseem merged 1 commit into
masterfrom
aliwaseem/fe-3882-downgrading-to-free-tier-with-an-active-branch-shows-a-brief-warning
Jul 9, 2026
Merged

fix(studio): extend downgrade error toast duration#47780
awaseem merged 1 commit into
masterfrom
aliwaseem/fe-3882-downgrading-to-free-tier-with-an-active-branch-shows-a-brief-warning

Conversation

@awaseem

@awaseem awaseem commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Downgrading to Free tier is blocked server-side when an org has an active branch, but the resulting error toast in ExitSurveyModal.tsx used the default 4s toast duration, making it easy to miss.
  • Adds duration: 10_000, dismissible: true, matching the pattern already used for other important billing error toasts (org-subscription-update-mutation.ts, NewOrgForm.tsx).

Fixes FE-3882

Test plan

  • Attempt to downgrade an org with an active branch to Free tier and confirm the error toast stays visible for 10s and can be dismissed

Summary by CodeRabbit

  • Bug Fixes
    • Improved the downgrade error message to stay visible longer and be easier to dismiss, making failures clearer for users.

The toast shown when a Free tier downgrade is blocked (e.g. active branch) auto-dismissed after the default 4s, making the failure easy to miss. Match the duration/dismissible pattern used elsewhere in the billing error flow.
@awaseem
awaseem requested a review from a team as a code owner July 9, 2026 13:13
@vercel

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
studio-self-hosted Ready Ready Preview, Comment Jul 9, 2026 1:18pm
studio-staging Ready Ready Preview, Comment Jul 9, 2026 1:18pm
5 Skipped Deployments
Project Deployment Actions Updated (UTC)
studio Ignored Ignored Jul 9, 2026 1:18pm
design-system Skipped Skipped Jul 9, 2026 1:18pm
docs Skipped Skipped Open in v0 Jul 9, 2026 1:18pm
ui-library Skipped Skipped Jul 9, 2026 1:18pm
zone-www-dot-com Skipped Skipped Open in v0 Jul 9, 2026 1:18pm

Request Review

@supabase

supabase Bot commented Jul 9, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 71d982a6-fdf0-4954-a8c6-b19c38b46c88

📥 Commits

Reviewing files that changed from the base of the PR and between 1987f19 and 2ce950a.

📒 Files selected for processing (1)
  • apps/studio/components/interfaces/Organization/BillingSettings/Subscription/ExitSurveyModal.tsx

📝 Walkthrough

Walkthrough

The downgrade mutation's error handler in ExitSurveyModal now calls toast.error with additional options: a 10-second duration and dismissible flag, replacing the previous default toast.error invocation.

Changes

Toast Error Options

Layer / File(s) Summary
Downgrade error toast configuration
apps/studio/components/interfaces/Organization/BillingSettings/Subscription/ExitSurveyModal.tsx
The onError handler's toast.error call now includes duration: 10000 and dismissible: true options instead of a plain message-only toast.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately describes the main change: extending the downgrade error toast duration.
Description check ✅ Passed The description covers the bug, fix, linked issue, and test plan, though it doesn't fully mirror the template headings.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch aliwaseem/fe-3882-downgrading-to-free-tier-with-an-active-branch-shows-a-brief-warning

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.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

🎭 Playwright Test Results

passed  237 passed
flaky  1 flaky
skipped  5 skipped

Details

stats  243 tests across 28 suites
duration  11 minutes, 9 seconds
commit  2ce950a

Flaky tests

Features › queue-table-operations.spec.ts › Queue Table Operations › pending changes persist when switching between tables

Skipped tests

Features › auth-users.spec.ts › should show web3 users as enabled when the matching web3 provider is enabled
Features › sql-editor.spec.ts › SQL Editor › snippet favourite works as expected
Features › sql-editor.spec.ts › SQL Editor › share with team works as expected
Features › sql-editor.spec.ts › SQL Editor › folders works as expected
Features › sql-editor.spec.ts › SQL Editor › other SQL snippets actions work as expected

@awaseem
awaseem merged commit d949a19 into master Jul 9, 2026
33 of 34 checks passed
@awaseem
awaseem deleted the aliwaseem/fe-3882-downgrading-to-free-tier-with-an-active-branch-shows-a-brief-warning branch July 9, 2026 13:58
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Braintrust eval report

Assistant (master-1783605651)

Score Average Improvements Regressions
Knowledge Usage 97.4% (+3pp) 1 🟢 1 🔴
SQL Identifier Quoting 100% (+0pp) - -
SQL Validity 100% (+0pp) - -
Tool Usage 76.5% (+4pp) 1 🟢 -
Time_to_first_token 2.9tok (-0.4tok) 17 🟢 12 🔴
Llm_calls 3.32 (+0.38) 18 🟢 8 🔴
Tool_calls 3.18 (+0.15) 12 🟢 8 🔴
Errors 5.23 (-0.07) 6 🟢 2 🔴
Llm_errors 0 (+0) - -
Tool_errors 0 (+0) - -
Prompt_tokens 25071.51tok (-1254.94tok) 11 🟢 14 🔴
Prompt_cached_tokens 7750.62tok (+111.82tok) 12 🟢 11 🔴
Prompt_cache_creation_tokens 0tok (+0tok) - -
Prompt_cache_creation_5m_tokens 0tok (+0tok) - -
Prompt_cache_creation_1h_tokens 0tok (+0tok) - -
Completion_tokens 604.07tok (+7.44tok) 15 🟢 14 🔴
Completion_reasoning_tokens 96.18tok (+5.66tok) 9 🟢 17 🔴
Total_tokens 25675.57tok (-1247.51tok) 12 🟢 17 🔴
Estimated_cost 0$ (0$) 11 🟢 15 🔴
Duration 17.93s (-0.69s) 16 🟢 13 🔴
Llm_duration 13.49s (+0.54s) 16 🟢 13 🔴

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