Skip to content

fix(website): correct top-up credit rollover copy on cloud pricing#12923

Merged
deepme987 merged 1 commit into
mainfrom
deepme987/website/topup-credits-no-rollover
Jun 17, 2026
Merged

fix(website): correct top-up credit rollover copy on cloud pricing#12923
deepme987 merged 1 commit into
mainfrom
deepme987/website/topup-credits-no-rollover

Conversation

@deepme987

Copy link
Copy Markdown
Contributor

Summary

The /cloud/pricing page's "Add more credits anytime" feature claims unused top-up credits roll over to the next month. Per product (Pablo, #cloud), top-up credits do not roll over. Fix the copy to match policy and the docs.

Changes

  • What: pricing.included.feature5.description (en + zh-CN) in apps/website/src/i18n/translations.ts.
    • Before: "Purchase additional credits at any time. Unused top-ups roll over to the next month automatically for up to 1 year."
    • After: "Purchase additional credits at any time. Top-up credits are valid for 1 year from the date of purchase and do not roll over with your monthly plan."
  • Keeps the accurate 1-year validity window (matches docs/interface/credits.mdx).
  • Scope: this is the only rollover claim on the page. The tier cards (lines 1224/1255/1286) only say "top-ups available" — no change needed.
  • Breaking: none (copy only).

Deliberately out of scope

The Slack thread surfaced an unresolved question: Pablo said top-ups don't roll over "as opposed to subscription credits which do," but docs/interface/credits.mdx says monthly credits don't roll over either. This PR makes no claim about subscription/monthly credit rollover (the new copy is neutral: "do not roll over with your monthly plan"). Resolving that — and the optional FAQ entry / docs alignment Glary-Bot proposed — needs a product decision first. Flagging for follow-up.

The 'Add more credits anytime' feature on /cloud/pricing claimed unused
top-up credits 'roll over to the next month automatically for up to 1
year.' Per product (Pablo, #cloud Slack) top-up credits do NOT roll over.
Reword (en + zh-CN) to state top-up credits are valid for 1 year from
purchase and do not roll over with the monthly plan, matching
docs/interface/credits.mdx. Scoped to the only rollover claim on the page;
the tier cards only say 'top-ups available' and need no change.
@coderabbitai

coderabbitai Bot commented Jun 17, 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: 1c050f8f-85db-4553-9204-fc78c1cf6ec7

📥 Commits

Reviewing files that changed from the base of the PR and between 6850d22 and bc65668.

📒 Files selected for processing (1)
  • apps/website/src/i18n/translations.ts

📝 Walkthrough

Walkthrough

Two translation strings (en and zh-CN) for pricing.included.feature5.description in translations.ts are updated to state that purchased top-up credits are valid for 1 year from the purchase date and do not roll over with the monthly plan, replacing prior wording about automatic monthly rollover.

Changes

Pricing Top-up Credit Copy

Layer / File(s) Summary
Top-up credit description strings
apps/website/src/i18n/translations.ts
Updates pricing.included.feature5.description in EN and zh-CN: replaces "unused top-ups auto-roll over to next month" with "top-ups valid for 1 year from purchase, do not roll over with monthly plan".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 Hop hop, a tiny text tweak,
Credits now last a year, not a week-to-week!
No more rolling over with the monthly tide,
Just purchase your top-ups and let them reside.
The bunny approves — the copy's clear and bright! ✨

🚥 Pre-merge checks | ✅ 7
✅ Passed checks (7 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: correcting copy about credit rollover behavior on the pricing page.
Description check ✅ Passed The description provides a clear summary, detailed before/after changes, breaks down the scope, and explicitly notes out-of-scope items for follow-up.
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.
End-To-End Regression Coverage For Fixes ✅ Passed PR changes are exclusively in apps/website (translations for pricing page copy). This directory is explicitly exempted from end-to-end regression test requirements in the check instructions.
Adr Compliance For Entity/Litegraph Changes ✅ Passed Check does not apply: PR only modifies apps/website/src/i18n/translations.ts, which is outside the required scopes (src/lib/litegraph/, src/ecs/, or graph entity files).

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

✨ 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 deepme987/website/topup-credits-no-rollover

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown

🌐 Website E2E

Caution

Some tests failed.

Status ❌ Failed
Report unavailable
Screenshot diffs 0
Other failures 0

🔗 Website Preview

Website Preview: https://comfy-website-preview-pr-12923.vercel.app

This commit: https://website-frontend-94xvmfmgb-comfyui.vercel.app

Last updated: 2026-06-17T17:52:27Z for bc65668

@deepme987 deepme987 marked this pull request as ready for review June 17, 2026 18:27
@deepme987 deepme987 requested a review from a team June 17, 2026 18:27
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 17, 2026
@deepme987 deepme987 added this pull request to the merge queue Jun 17, 2026
Merged via the queue into main with commit ccf2f12 Jun 17, 2026
48 of 49 checks passed
@deepme987 deepme987 deleted the deepme987/website/topup-credits-no-rollover branch June 17, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants