fix: update billing docs with new billing interval information#672
Open
tamalchowdhury wants to merge 2 commits intomainfrom
Open
fix: update billing docs with new billing interval information#672tamalchowdhury wants to merge 2 commits intomainfrom
tamalchowdhury wants to merge 2 commits intomainfrom
Conversation
Contributor
WalkthroughThis PR updates six billing docs to replace monthly-only subscription wording with configurable billing intervals (daily, weekly, monthly, yearly), adds an optional annual pricing offering for fixed charges, and introduces/clarifies related billing terminology. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (3)
src/content/docs/billing/manage-plans/about-plans.mdx (1)
82-82: Consider clarifying the yearly pricing format in the table.The table shows
$ 20 / month or yearlybut doesn't indicate that yearly billing uses a custom annual price (as documented elsewhere). This could confuse readers who expect the yearly price to simply be$20 × 12.✏️ Suggested improvement
-| Subscription | $ 20 / month or yearly | Fixed charge | +| Subscription | $20/month (or custom yearly) | Fixed charge |🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/content/docs/billing/manage-plans/about-plans.mdx` at line 82, Update the table cell that currently reads " $ 20 / month or yearly" to explicitly indicate that yearly billing uses a custom annual price rather than simply 12× the monthly rate; for example, change the Subscription column entry (the row with heading "Subscription") to something like "$20 / month or custom annual price for yearly billing" or "$20 / month or a custom yearly price (not simply $20×12)" so readers understand the yearly charge is documented elsewhere as a custom annual price.src/content/docs/billing/pricing/pricing-models.mdx (1)
33-33: Minor: Avoid repetitive "yearly" in the same sentence.The phrase "custom yearly price when the customer selects yearly billing" uses "yearly" twice in close proximity, which reads awkwardly.
✏️ Suggested rewording
-In Kinde, create a **Fixed charge** and call it something like `Subscription fee`. Choose a **billing interval** (daily, weekly, monthly, or yearly) and set the price. You can optionally **offer as annual**—bill the charge once per year at a custom yearly price when the customer selects yearly billing (only one fixed charge per plan can be offered as annual). Re-use the charge in each plan and change the price and line item description as needed. +In Kinde, create a **Fixed charge** and call it something like `Subscription fee`. Choose a **billing interval** (daily, weekly, monthly, or yearly) and set the price. You can optionally **offer as annual**—bill the charge once per year at a custom price when the customer selects annual billing (only one fixed charge per plan can be offered as annual). Re-use the charge in each plan and change the price and line item description as needed.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/content/docs/billing/pricing/pricing-models.mdx` at line 33, The sentence in the pricing-models.mdx diff repeats "yearly" ("custom yearly price when the customer selects yearly billing"); replace that phrase with a non-repetitive alternative such as "custom annual price when the customer selects yearly billing" (or "custom yearly price when the customer selects annual billing") to remove the duplicate word while preserving meaning; update the sentence in the same paragraph where the Fixed charge / Subscription fee description appears.src/content/docs/billing/manage-plans/create-plans.mdx (1)
84-84: Minor: Same "yearly" repetition as in pricing-models.mdx.Consider rewording to avoid using "yearly" twice in close proximity.
✏️ Suggested rewording
-A fixed charge is a recurring charge such as base price or subscription fee. You can set **billing intervals** (daily, weekly, monthly, or yearly) and optionally **offer the charge as annual** with a custom yearly price when customers choose yearly billing. +A fixed charge is a recurring charge such as base price or subscription fee. You can set **billing intervals** (daily, weekly, monthly, or yearly) and optionally **offer the charge as annual** with a custom price when customers choose annual billing.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/content/docs/billing/manage-plans/create-plans.mdx` at line 84, The sentence "A fixed charge is a recurring charge..." repeats "yearly" twice; edit that sentence in create-plans.mdx to reword the second occurrence for clarity (for example change "when customers choose yearly billing" to "when customers choose annual billing" or "when customers opt for an annual plan") so it no longer uses "yearly" twice while preserving the explanation of billing intervals and the option to offer a custom yearly price; update the phrase near "billing intervals" / "offer the charge as annual" accordingly.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/content/docs/billing/about-billing/about-billing.mdx`:
- Around line 53-55: The bullet under the "Known limitations" section is
ambiguous about custom billing intervals; update the copy so it clearly states
whether custom intervals are supported or restricted. Either move the sentence
into the supported-features list and rephrase to: "Fixed charges support daily,
weekly, monthly, and yearly intervals (annual pricing available); custom
intervals such as quarterly are not supported," OR keep it under Known
limitations and reword to explicitly say: "Limitation: only fixed charges
support daily, weekly, monthly, and yearly intervals; custom intervals (e.g.,
quarterly) are not supported." Edit the line referencing "Custom billing
intervals (e.g., quarterly)..." accordingly and ensure consistency with
surrounding headings ("Known limitations" / supported features).
---
Nitpick comments:
In `@src/content/docs/billing/manage-plans/about-plans.mdx`:
- Line 82: Update the table cell that currently reads " $ 20 / month or yearly"
to explicitly indicate that yearly billing uses a custom annual price rather
than simply 12× the monthly rate; for example, change the Subscription column
entry (the row with heading "Subscription") to something like "$20 / month or
custom annual price for yearly billing" or "$20 / month or a custom yearly price
(not simply $20×12)" so readers understand the yearly charge is documented
elsewhere as a custom annual price.
In `@src/content/docs/billing/manage-plans/create-plans.mdx`:
- Line 84: The sentence "A fixed charge is a recurring charge..." repeats
"yearly" twice; edit that sentence in create-plans.mdx to reword the second
occurrence for clarity (for example change "when customers choose yearly
billing" to "when customers choose annual billing" or "when customers opt for an
annual plan") so it no longer uses "yearly" twice while preserving the
explanation of billing intervals and the option to offer a custom yearly price;
update the phrase near "billing intervals" / "offer the charge as annual"
accordingly.
In `@src/content/docs/billing/pricing/pricing-models.mdx`:
- Line 33: The sentence in the pricing-models.mdx diff repeats "yearly" ("custom
yearly price when the customer selects yearly billing"); replace that phrase
with a non-repetitive alternative such as "custom annual price when the customer
selects yearly billing" (or "custom yearly price when the customer selects
annual billing") to remove the duplicate word while preserving meaning; update
the sentence in the same paragraph where the Fixed charge / Subscription fee
description appears.
ℹ️ Review info
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
src/content/docs/billing/about-billing/about-billing.mdxsrc/content/docs/billing/about-billing/billing-concepts-terms.mdxsrc/content/docs/billing/about-billing/kinde-billing-model.mdxsrc/content/docs/billing/manage-plans/about-plans.mdxsrc/content/docs/billing/manage-plans/create-plans.mdxsrc/content/docs/billing/pricing/pricing-models.mdx
Deploying kinde-docs-preview with
|
| Latest commit: |
e24e7cb
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://5d15f1f8.kinde-docs-preview.pages.dev |
| Branch Preview URL: | https://tamal-update-billing-interva.kinde-docs-preview.pages.dev |
tamalchowdhury
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description (required)
Related issues & labels (optional)
Summary by CodeRabbit