Repository navigation
Show Business limits as Custom, matching Enterprise, instead of TBD - #122
Conversation
|
All clear! No issues remaining. 🎉 1 issue already resolved
All required checks passed on
Select any unchecked box below to run or rerun that agent. Full resultsIndent Review Agent
|
|
Caution Review failedThe pull request is closed. Walkthrough
Merge Risk:
|
Codex Review Summary: ❌ FailedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
# Conflicts: # scripts/pricing-next.mjs
Business had TBD in every limit row Pro meters. Each of those rows now matches Enterprise, with Enterprise's "Contract" shown as "Custom" for Business. Business keeps its own published prices for compute, MySQL compute, Workers requests and compute, storage, members and included usage.
Concrete cells that now match Enterprise instead of TBD: Disk per Space "Up to 1 TB", memory per request "2 GB", build timeout "2 h", logs "Custom, plus log drains". Everything else is "Custom". Advanced crons and Extra email a day were TBD overrides; they are now Custom like Enterprise.
Deleted: the TBD constant, the TBD badge branch and
.sf-tag-tbdstyle incomponents/PricingTable.astro, and the TBD row in the fine print.node --test scripts/pricing.test.mjspasses 3/3, andbun run checkpasses with no type errors.Stacked on #121.
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.Summary by CodeRabbit