From 09fb69851cd834c3d138850da7830076c7e97b51 Mon Sep 17 00:00:00 2001
From: Raquel Smith <raquelmsmith@users.noreply.github.com>
Date: Thu, 28 Mar 2024 20:30:54 -0700
Subject: [PATCH] hide from plan comparison and don't include in upgrade link

---
 frontend/src/scenes/billing/PlanComparison.tsx | 1 +
 frontend/src/scenes/billing/billing-utils.ts   | 6 +++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/frontend/src/scenes/billing/PlanComparison.tsx b/frontend/src/scenes/billing/PlanComparison.tsx
index a185b4df0277f..85f24a825be73 100644
--- a/frontend/src/scenes/billing/PlanComparison.tsx
+++ b/frontend/src/scenes/billing/PlanComparison.tsx
@@ -288,6 +288,7 @@ export const PlanComparison = ({
                 )}
                 {includeAddons &&
                     product.addons?.map((addon) => {
+                        // TODO: integrated_identities addon will show up here when we add a price plan. Make sure this can handle it.
                         return addon.tiered ? (
                             <tr key={addon.name + 'pricing-row'} className="PlanTable__tr__border">
                                 <th scope="row">
diff --git a/frontend/src/scenes/billing/billing-utils.ts b/frontend/src/scenes/billing/billing-utils.ts
index b6b152c099fe4..bf690d0174a0c 100644
--- a/frontend/src/scenes/billing/billing-utils.ts
+++ b/frontend/src/scenes/billing/billing-utils.ts
@@ -169,7 +169,11 @@ export const getUpgradeProductLink = (
     url += `${product.type}:${upgradeToPlanKey},`
     if (includeAddons && product.addons?.length) {
         for (const addon of product.addons) {
-            if (addon.plans?.[0]?.plan_key) {
+            if (
+                // TODO: this breaks if we support multiple plans per addon due to just grabbing the first plan
+                addon.plans?.[0]?.plan_key &&
+                !addon.inclusion_only
+            ) {
                 url += `${addon.type}:${addon.plans[0].plan_key},`
             }
         }