Skip to content

Comments

chore: change primary button for resumes#42436

Merged
kevcodez merged 1 commit intomasterfrom
resume-button-primary
Feb 4, 2026
Merged

chore: change primary button for resumes#42436
kevcodez merged 1 commit intomasterfrom
resume-button-primary

Conversation

@kevcodez
Copy link
Contributor

@kevcodez kevcodez commented Feb 4, 2026

Currently seems to be leading to accidental upgrades+support tickets, so we are making the resume project the primary button instead of the upgrade

Screenshot 2026-02-04 at 9 36 22 AM

Summary by CodeRabbit

  • New Features

    • Added plan-specific actions in project paused state: Free plan users see "Upgrade to Pro" option with billing link, while paid plan users see "View project settings" option.
  • Style

    • Updated Resume button to use primary styling for better visual prominence.

Currently seems to be leading to accidental upgrades+support tickets, so we are making the resume project the primary button instead of the upgrade
@kevcodez kevcodez requested a review from a team as a code owner February 4, 2026 04:08
@vercel
Copy link

vercel bot commented Feb 4, 2026

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

Project Deployment Actions Updated (UTC)
studio-self-hosted Ready Ready Preview, Comment Feb 4, 2026 4:11am
studio-staging Ready Ready Preview, Comment Feb 4, 2026 4:11am
6 Skipped Deployments
Project Deployment Actions Updated (UTC)
cms Ignored Ignored Feb 4, 2026 4:11am
studio Ignored Ignored Feb 4, 2026 4:11am
design-system Skipped Skipped Feb 4, 2026 4:11am
docs Skipped Skipped Feb 4, 2026 4:11am
ui-library Skipped Skipped Feb 4, 2026 4:11am
zone-www-dot-com Skipped Skipped Feb 4, 2026 4:11am

Request Review

@supabase
Copy link

supabase bot commented Feb 4, 2026

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
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

Walkthrough

The paused project state footer UI was reorganized to conditionally display plan-specific action buttons (Upgrade to Pro for free plans, View project settings for paid plans) before the Resume button. The Resume button styling was updated to primary, and duplicate button rendering logic was consolidated.

Changes

Cohort / File(s) Summary
Paused State UI
apps/studio/components/layouts/ProjectLayout/PausedState/ProjectPausedState.tsx
Reorganized footer button layout with conditional rendering based on plan type. Moved action buttons before Resume button and updated Resume button to primary styling. Removed redundant duplicate button rendering.

Possibly related PRs

  • supabase/supabase#41300: Modifies the same paused state component file, touching the resume flow and button/modal rendering logic.

Suggested reviewers

  • joshenlim
  • alaister
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description lacks required template sections including 'kind of change', 'current behavior', 'new behavior', and confirmation of reading CONTRIBUTING.md. Add missing template sections: confirm reading CONTRIBUTING.md, specify change type (UI/behavior fix), describe current problematic behavior, and explain new behavior clearly.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title refers to changing the primary button styling for resume functionality, which is the main technical change in this PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch resume-button-primary

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.

@coveralls
Copy link

Coverage Status

coverage: 66.278%. remained the same
when pulling 64e0370 on resume-button-primary
into 3bf3750 on master.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

🎭 Playwright Test Results

passed  86 passed
skipped  4 skipped

Details

stats  90 tests across 14 suites
duration  10 minutes, 44 seconds
commit  64e0370

Skipped tests

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

@kevcodez kevcodez merged commit 4c9e5cf into master Feb 4, 2026
25 checks passed
@kevcodez kevcodez deleted the resume-button-primary branch February 4, 2026 05:25
@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Braintrust eval report

Assistant (master-1770182861)

Score Average Improvements Regressions
Completeness 100% (+0pp) - -
Conciseness 0% (+0pp) - -
Goal Completion 95.8% (-4pp) - 1 🔴
SQL Identifier Quoting 100% (+0pp) - -
SQL Validity 100% (+0pp) - -
Tool Usage 100% (+0pp) - -
Correctness 100% (+0pp) - -
Docs Faithfulness 50% (+0pp) - 1 🔴
URL Validity 100% (+0pp) - -
Time_to_first_token 0.17tok (-0.04tok) 9 🟢 3 🔴
Llm_calls 7.92 (-0.5) 1 🟢 5 🔴
Tool_calls 2.5 (-0.58) - 3 🔴
Errors 0 (+0) - -
Llm_errors 0 (+0) - -
Tool_errors 0 (+0) - -
Prompt_tokens 85175.17tok (-9605.33tok) 8 🟢 4 🔴
Prompt_cached_tokens 48554.67tok (+11754.67tok) 5 🟢 5 🔴
Prompt_cache_creation_tokens 0tok (+0tok) - -
Completion_tokens 5613tok (+649.92tok) 3 🟢 9 🔴
Completion_reasoning_tokens 3936tok (+522.67tok) 3 🟢 8 🔴
Completion_accepted_prediction_tokens 0tok (+0tok) - -
Completion_rejected_prediction_tokens 0tok (+0tok) - -
Completion_audio_tokens 0tok (+0tok) - -
Total_tokens 90788.17tok (-8955.42tok) 8 🟢 4 🔴
Estimated_cost 0.02$ (0$) 6 🟢 5 🔴
Duration 40.41s (+3.04s) 4 🟢 8 🔴
Llm_duration 80.08s (+9.1s) 4 🟢 8 🔴

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.

3 participants