feat(Billing): Reintroduce the Scale-Up plan#7216
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Docker builds report
|
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
|
Visual Regression16 screenshots compared. See report for details. |
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
frontend/web/components/pages/organisation-settings/tabs/BillingTab.tsx
Outdated
Show resolved
Hide resolved
frontend/web/components/pages/organisation-settings/tabs/BillingTab.tsx
Outdated
Show resolved
Hide resolved
talissoncosta
left a comment
There was a problem hiding this comment.
Just some minor comments regarding the imports... Otherwise looks good.
Have you tested the upgrade flow end-to-end with rtk_payment_modal_migration both on and off?
The new payment modal is behind the flag, but the getNextPlan() change in utils.tsx isn't — it affects the upgrade chips across the app for everyone. Users on the legacy modal would see "Upgrade to Scale-Up" chips but land on a modal that doesn't have a Scale-Up plan.
The flag has been on for internal testing for a while — could be a good opportunity to enable it for everyone alongside this. Otherwise we should gate getNextPlan() behind the flag to keep the old behaviour for legacy modal users.
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
talissoncosta
left a comment
There was a problem hiding this comment.
All good! Thanks for handling that!
|
@talissoncosta Thanks for the great review here. I've started from scratch, aiming to support both old and new UIs. I've tested both, but haven't gone through activating a trial — I trust the billing masters above us know what they're doing. 👀 I'll give it a go once live. |
I have added information todocs/if required so people know about the feature.Changes
Contributes to https://github.com/Flagsmith/flagsmith-private/issues/107
Restores the Scale-Up plan in the in-app organisation billing page.
Also includes a fix to make "Startup" the first upgrade plan from free.
How did you test this code?