From 868dd5f2203c3e80088641f14388a2fbe908c4fd Mon Sep 17 00:00:00 2001
From: Kian Newman-Hazel <kian@cloudflare.com>
Date: Mon, 16 Dec 2024 17:58:54 +0000
Subject: [PATCH] [Docs Site] Add check for plan-level info in ProductFeatures

---
 src/components/ProductFeatures.astro | 104 +++++++++++++++++----------
 1 file changed, 65 insertions(+), 39 deletions(-)

diff --git a/src/components/ProductFeatures.astro b/src/components/ProductFeatures.astro
index 73ad198d54e3367..f3ef124eab8bb75 100644
--- a/src/components/ProductFeatures.astro
+++ b/src/components/ProductFeatures.astro
@@ -38,7 +38,7 @@ const entries = Object.entries(plan);
 						<a href={value.link}>{value.title}</a>
 					</p>
 				)}
-				{Object.entries(value.properties).map(([_, value]: [string, any]) => (
+				{Object.values(value.properties).map((value: any) => (
 					<p>
 						<strong
 							set:html={marked.parseInline(
@@ -51,52 +51,78 @@ const entries = Object.entries(plan);
 							<p>{value.summary}</p>
 						)}
 						<ul>
-							<li>
-								<strong>Free: </strong>
-								<Fragment
-									set:html={marked.parseInline(value.free.toString())}
-								/>
-							</li>
+							{value.free && (
+								<li>
+									<strong>Free: </strong>
+									<Fragment
+										set:html={marked.parseInline(value.free.toString())}
+									/>
+								</li>
+							)}
 							{additional_descriptions && (
+								<>
+									{value.lite ||
+										(value.free && (
+											<li>
+												<strong>Lite: </strong>
+												{value.lite ? (
+													<Fragment
+														set:html={marked.parseInline(value.lite.toString())}
+													/>
+												) : (
+													<Fragment
+														set:html={marked.parseInline(value.free.toString())}
+													/>
+												)}
+											</li>
+										))}
+								</>
+							)}
+							{value.pro && (
 								<li>
-									<strong>Lite: </strong>
-									{value.lite ? (
-										<Fragment
-											set:html={marked.parseInline(value.lite.toString())}
-										/>
-									) : (
-										<Fragment
-											set:html={marked.parseInline(value.free.toString())}
-										/>
-									)}
+									<strong>Pro: </strong>
+									<Fragment
+										set:html={marked.parseInline(value.pro.toString())}
+									/>
 								</li>
 							)}
-							<li>
-								<strong>Pro: </strong>
-								<Fragment set:html={marked.parseInline(value.pro.toString())} />
-							</li>
 							{additional_descriptions && (
+								<>
+									{value.pro_plus ||
+										(value.pro && (
+											<li>
+												<strong>Pro Plus: </strong>
+												{value.pro_plus ? (
+													<Fragment
+														set:html={marked.parseInline(
+															value.pro_plus.toString(),
+														)}
+													/>
+												) : (
+													<Fragment
+														set:html={marked.parseInline(value.pro.toString())}
+													/>
+												)}
+											</li>
+										))}
+								</>
+							)}
+							{value.biz && (
+								<li>
+									<strong>Business: </strong>
+									<Fragment
+										set:html={marked.parseInline(value.biz.toString())}
+									/>
+								</li>
+							)}
+							{value.ent && (
 								<li>
-									<strong>Pro Plus: </strong>
-									{value.pro_plus ? (
-										<Fragment
-											set:html={marked.parseInline(value.pro_plus.toString())}
-										/>
-									) : (
-										<Fragment
-											set:html={marked.parseInline(value.pro.toString())}
-										/>
-									)}
+									<strong>Enterprise: </strong>
+									<Fragment
+										set:html={marked.parseInline(value.ent.toString())}
+									/>
 								</li>
 							)}
-							<li>
-								<strong>Business: </strong>
-								<Fragment set:html={marked.parseInline(value.biz.toString())} />
-							</li>
-							<li>
-								<strong>Enterprise: </strong>
-								<Fragment set:html={marked.parseInline(value.ent.toString())} />
-							</li>
 						</ul>
 					</p>
 				))}