-
Notifications
You must be signed in to change notification settings - Fork 36
fix: Allow yearly plan upgrades if on yearly plan #3965
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #3965 +/- ##
==========================================
- Coverage 98.73% 98.68% -0.06%
==========================================
Files 826 826
Lines 14983 15085 +102
Branches 4287 4359 +72
==========================================
+ Hits 14793 14886 +93
- Misses 182 191 +9
Partials 8 8
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #3965 +/- ##
==========================================
- Coverage 98.73% 98.68% -0.06%
==========================================
Files 826 826
Lines 14983 15085 +102
Branches 4287 4351 +64
==========================================
+ Hits 14793 14886 +93
- Misses 182 191 +9
Partials 8 8
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
✅ Deploy preview for gazebo ready!Previews expire after 1 month automatically.
|
Codecov Report❌ Patch coverage is @@ Coverage Diff @@
## main #3965 +/- ##
==========================================
+ Coverage 95.34% 98.68% +3.33%
==========================================
Files 826 826
Lines 14983 15085 +102
Branches 4287 4351 +64
==========================================
+ Hits 14285 14886 +601
+ Misses 618 191 -427
+ Partials 80 8 -72
... and 53 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Bundle ReportChanges will increase total bundle size by 8.71kB (0.07%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: gazebo-production-systemAssets Changed:
Files in
Files in
view changes for bundle: gazebo-production-esmAssets Changed:
Files in
Files in
|
Closes https://linear.app/getsentry/issue/CCMRG-2016/allow-yearly-plan-upgrades-for-current-customers
Changes made on top of #3958
Allows yearly upgrades if the customer is currently on a yearly plan. Otherwise, yearly plans should still continue to not be shown as an option.
Note
Enables annual billing selection for existing annual subscribers and keeps plan/billing choices consistent across the upgrade flow.
ANNUALradio option inBillingOptions(Pro/Sentry/Team) only when current plan is annual; persist selection viasetFormValue/setSelectedPlanand local statePriceCallout(Pro/Sentry/Team) to render annual per-month price, annual total, and savings (with Sentry-specific diff check)PlanTypeOptions) and when computing defaults (determineDefaultPlan); include yearly variants infind*PlansnewPlanthrough controllers to pricing components; display dynamic base unit price/billing rate textWritten by Cursor Bugbot for commit 2938dbb. This will update automatically on new commits. Configure here.