more info

[ildyria]

Summary by CodeRabbit

• New Features

  • Added a reusable "Pro" badge and updated the features table to visually mark Pro-only capabilities.
  • Supporter Edition page now segregates Pro-only webshop/payment features and reflects those changes in the pricing/feature listings.

• Documentation

  • Expanded FAQ with Pro edition details and guidance for retrieving and managing license keys.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Warning

Rate limit exceeded

@ildyria has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 23 minutes and 0 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 5e25a66 and 0f18828.

📒 Files selected for processing (1)

• src/pages/get-supporter-edition.astro (6 hunks)

📝 Walkthrough

Walkthrough

Adds a new inline Pro badge component, an optional isPro flag on Spec, updates feature-list rendering to show Pro indicators/styling, and converts several Supporter Edition pricing/features and FAQs to mark and describe Pro-only items.

Changes

Cohort / File(s)	Summary
New UI component src/components/ui/Pro.astro	Adds a small, rounded inline "Pro" badge component that imports twMerge, accepts tag (default "Pro") and classes, and merges base Tailwind classes with user classes for rendering.
Type update src/types.d.ts	Adds optional isPro?: boolean to the Spec interface.
Feature rendering logic src/components/widgets/FeaturesList.astro	Imports the Pro component, reads isPro from specs, renders <Pro/> in header cells for Pro features, and applies amber-themed classes to Tick elements for Pro availability in data rows.
Content & pricing page src/pages/get-supporter-edition.astro	Replaces non‑Pro webshop features with a Pro-only webshop section (sets isPro: true on related specs), adds PayPal as a Pro entry, updates pricing text and footnotes to reference Pro, and adds/edits FAQs about the Pro edition and license key retrieval/lifetime licenses.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

• Extra attention:
  • src/pages/get-supporter-edition.astro — verify content consistency, footnote markers, and that moved features are correctly flagged isPro.
  • src/components/widgets/FeaturesList.astro — check conditional rendering paths, Tick styling, and accessibility/spacing with the new <Pro/> badge.
  • src/components/ui/Pro.astro — confirm twMerge import and class merging yield expected class order and no unintended overrides.
  • src/types.d.ts — ensure all usages of Spec safely handle the optional isPro field.

Poem

🐰 I stitched a tiny amber star, a Pro to wear with pride,
It hops beside the headers where the premium bits reside.
A little badge that winks and gleams, it burrows in the lists,
With carrots, keys, and FAQ dreams — I celebrate with twigs and fists! 🥕✨

Pre-merge checks

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 2

🧹 Nitpick comments (1)

src/components/ui/Pro.astro (1)

1-7: Consider adding TypeScript prop types for better type safety.

The component works correctly, but adding an interface for props would improve maintainability and IDE support.

🔎 Optional: Add prop types

 ---
 import { twMerge } from 'tailwind-merge';
 
+interface Props {
+  tag?: string;
+  classes?: string;
+}
+
-const { tag = 'Pro', classes = {} } = Astro.props;
+const { tag = 'Pro', classes = '' } = Astro.props;
 ---
 
 <span class={twMerge("rounded-full text-xs text-white bg-amber-600 dark:text-neutral-200 dark:bg-amber-800 px-2 shadow-sm shadow-black/20 ml-2", classes)}><Fragment set:html={tag} /></span>

Note: The classes default should likely be an empty string '' rather than {} since twMerge expects string arguments. Currently this works because twMerge coerces the empty object, but it's not type-safe.

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c7d91f1 and d44791b.

📒 Files selected for processing (4)

• src/components/ui/Pro.astro (1 hunks)
• src/components/widgets/FeaturesList.astro (2 hunks)
• src/pages/get-supporter-edition.astro (6 hunks)
• src/types.d.ts (1 hunks)

🔇 Additional comments (3)

src/types.d.ts (1)

257-265: LGTM!

The isPro optional boolean property follows the existing pattern of feature flags (isHeader, isV6, isSoon) in the Spec interface and integrates well with the component usage in FeaturesList.astro.

src/components/widgets/FeaturesList.astro (1)

42-49: LGTM!

The implementation cleanly extends the feature list to support Pro-specific styling:

• Proper destructuring of isPro from specs
• Pro badge appended to headers when applicable
• Amber-themed Tick styling for Pro features provides good visual distinction matching the Pro badge color scheme

src/pages/get-supporter-edition.astro (1)

544-579: LGTM!

The Pro/Webshop integration section is well-structured:

• Header properly marked with both isSoon and isPro flags
• All child features consistently marked with isPro: true
• Availability array [false, true] correctly indicates these are SE-exclusive features

[coderabbitai]

Actionable comments posted: 1

♻️ Duplicate comments (2)

src/pages/get-supporter-edition.astro (2)

599-604: Typos remain unfixed from previous review.

The typos "Depending of" → "Depending on" and missing "a" before "few" were flagged in a previous review and still need to be addressed.

---

651-660: Typos remain unfixed from previous review.

The typos "reccords" → "records" and "discord" → "Discord" were flagged in a previous review and still need to be addressed.

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d44791b and af789a9.

📒 Files selected for processing (1)

• src/pages/get-supporter-edition.astro (6 hunks)

🔇 Additional comments (3)

src/pages/get-supporter-edition.astro (3)

121-121: LGTM!

Formatting change removing trailing comma is fine.

---

543-584: LGTM!

The Pro features section is well-structured with consistent use of the isPro flag. The footnote reference (****) correctly links to the explanation in the extra text.

---

619-622: LGTM!

The updated text clearly explains the license key retrieval process.