From 22799fe56e0a4b26235ca8714eddfb1496b19972 Mon Sep 17 00:00:00 2001
From: Benjamin Goering <171782+gobengo@users.noreply.github.com>
Date: Thu, 9 Nov 2023 15:58:30 -0800
Subject: [PATCH] feat: /account/payment disables changing plans after
 scheduled sunset date (#2321)

Motivation:
* https://github.com/web3-storage/secrets/issues/23

Notes
* it takes effect when rendering after the date configured by
`NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_START`, which is also the date that is
used inside the announcement banner. We can easily change the exact
logic in the `shouldPreventPlanSwitching` in w3up-launch.js later too
* I got sick of looking at a React warning in browser console about
invalid nesting of div inside p, and it was because payment page Tooltip
(div) was in a p, so I made the Tooltip element name configurable to get
rid of the warning. seems to work ok
---
 .env.tpl                                      |  4 ++
 .github/workflows/website.yml                 |  2 +
 .../account/paymentTable.js/paymentTable.js   | 43 +++++++++++++++----
 packages/website/components/w3up-launch.js    | 11 ++++-
 .../zero/components/tooltip/tooltip.js        |  9 ++--
 packages/website/pages/account/payment.js     |  7 ++-
 6 files changed, 61 insertions(+), 15 deletions(-)

diff --git a/.env.tpl b/.env.tpl
index 52463255c1..98c39973a8 100644
--- a/.env.tpl
+++ b/.env.tpl
@@ -73,6 +73,10 @@ NEXT_PUBLIC_ENV=dev
 # NEXT_PUBLIC_COUNTLY_KEY=
 # NEXT_PUBLIC_COUNTLY_URL=
 
+# set to schedule when announcement banners should show about the w3up launch and related product sunsets
+# NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_ANNOUNCEMENT_START='2023-11-21T00:00:00Z'
+# set to schedule when certain features hide bcause the product has been sunset and w3up is launched
+# NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_START='2024-01-10T00:00:00Z'
 
 ## ---- cron ------------------------------------------------------------------
 
diff --git a/.github/workflows/website.yml b/.github/workflows/website.yml
index ef3900a8dc..19a1e47dcf 100644
--- a/.github/workflows/website.yml
+++ b/.github/workflows/website.yml
@@ -200,6 +200,7 @@ jobs:
           NEXT_PUBLIC_COUNTLY_KEY: ${{ secrets.COUNTLY_KEY }}
           NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY: ${{ secrets.TESTING_STRIPE_PUBLISHABLE_KEY }}
           NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_ANNOUNCEMENT_START: ${{ vars.NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_ANNOUNCEMENT_START }}
+          NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_START: ${{ vars.NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_START }}
           DID_DOCUMENT_ID: ${{ secrets.STAGING_DID_DOCUMENT_ID }}
           DID_DOCUMENT_PRIMARY_DID_KEY: ${{ secrets.STAGING_DID_DOCUMENT_PRIMARY_DID_KEY }}
       - name: Add to web3.storage
@@ -296,6 +297,7 @@ jobs:
           NEXT_PUBLIC_COUNTLY_KEY: ${{ secrets.COUNTLY_KEY }}
           NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY: ${{ secrets.STRIPE_PUBLISHABLE_KEY }}
           NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_ANNOUNCEMENT_START: ${{ vars.NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_ANNOUNCEMENT_START }}
+          NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_START: ${{ vars.NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_START }}
           DID_DOCUMENT_ID: ${{ secrets.PRODUCTION_DID_DOCUMENT_ID }}
           DID_DOCUMENT_PRIMARY_DID_KEY: ${{ secrets.PRODUCTION_DID_DOCUMENT_PRIMARY_DID_KEY }}
       - name: Add to web3.storage
diff --git a/packages/website/components/account/paymentTable.js/paymentTable.js b/packages/website/components/account/paymentTable.js/paymentTable.js
index e5994f8fe6..7b5489c487 100644
--- a/packages/website/components/account/paymentTable.js/paymentTable.js
+++ b/packages/website/components/account/paymentTable.js/paymentTable.js
@@ -7,17 +7,37 @@ import { formatAsStorageAmount, formatCurrency } from '../../../lib/utils.js';
 
 const getAdditionalStoragePrice = price => (typeof price === 'number' ? `${formatCurrency(price / 100)} / GiB` : 'N/A');
 
-const PaymentTable = ({ plans: plansProp, currentPlan, setPlanSelection, setIsPaymentPlanModalOpen }) => {
+/**
+ * @typedef {import('../../../components/contexts/plansContext.js').Plan} Plan
+ * @typedef {import('../../../components/contexts/plansContext.js').StorageSubscription} StorageSubscription
+ * @typedef {import('../../../components/contexts/plansContext.js').StoragePrice} StoragePrice
+ */
+
+/**
+ * @param {object} props
+ * @param {Plan} [props.currentPlan]
+ * @param {boolean} [props.disablePlanSwitching] - whether to disable plan switching e.g. hide 'select plan' buttons
+ * @param {Plan[]} props.plans
+ * @param {(isOpen: boolean) => void} props.setIsPaymentPlanModalOpen
+ * @param {(plan?: Plan) => void} props.setPlanSelection
+ */
+const PaymentTable = ({
+  plans: plansProp,
+  currentPlan,
+  setPlanSelection,
+  setIsPaymentPlanModalOpen,
+  disablePlanSwitching,
+}) => {
   const plans = useMemo(() => {
-    const isCurrentPlanStandard = ['free', 'lite', 'pro'].includes(currentPlan?.id);
+    const isCurrentPlanStandard = currentPlan ? ['free', 'lite', 'pro'].includes(currentPlan?.id) : false;
 
     if (!isCurrentPlanStandard) {
-      return [currentPlan, ...plansProp.slice(1)];
+      return [...(currentPlan ? [currentPlan] : []), ...plansProp.slice(1)];
     }
 
     return plansProp;
   }, [plansProp, currentPlan]);
-
+  const shouldShowSelectPlanButtons = !disablePlanSwitching;
   return (
     <>
       {currentPlan && (
@@ -42,7 +62,10 @@ const PaymentTable = ({ plans: plansProp, currentPlan, setPlanSelection, setIsPa
                 <p>Base Storage Capacity</p>
                 <p>
                   Additional Storage{' '}
-                  <Tooltip content="This is a charge for storage use above your limit. Please refer to <a href='/terms' target='_blank'>Terms of Service</a> for more information.">
+                  <Tooltip
+                    content="This is a charge for storage use above your limit. Please refer to <a href='/terms' target='_blank'>Terms of Service</a> for more information."
+                    element="span"
+                  >
                     <InfoIcon />
                   </Tooltip>
                 </p>
@@ -62,19 +85,21 @@ const PaymentTable = ({ plans: plansProp, currentPlan, setPlanSelection, setIsPa
                     </div>
                   </div>
 
-                  {!currentPlan?.tiers?.length && plan.id === currentPlan.id ? (
+                  {!currentPlan?.tiers?.length && plan.id === currentPlan?.id ? (
                     <div className="billing-plan-details">
                       <p className="preferred-desc">{currentPlan.description}</p>
                     </div>
                   ) : (
                     <div className="billing-plan-details">
-                      <p>{formatAsStorageAmount(plan.tiers?.[0]?.upTo)}</p>
+                      <p>{formatAsStorageAmount(plan.tiers?.[0]?.upTo ?? 0)}</p>
                       <p>{getAdditionalStoragePrice(plan.tiers?.[1]?.unitAmount)}</p>
-                      <p>{plan.bandwidth ? `${formatAsStorageAmount(plan.bandwidth)} / month` : 'N/A'}</p>
+                      <p>
+                        {plan.bandwidth ? `${formatAsStorageAmount(parseInt(plan.bandwidth, 10) ?? 0)} / month` : 'N/A'}
+                      </p>
                     </div>
                   )}
 
-                  {currentPlan?.id !== plan.id && (
+                  {shouldShowSelectPlanButtons && currentPlan?.id !== plan.id && (
                     <Button
                       variant="light"
                       disabled={currentPlan?.isPreferred}
diff --git a/packages/website/components/w3up-launch.js b/packages/website/components/w3up-launch.js
index 424f91aee4..e355c1e2f9 100644
--- a/packages/website/components/w3up-launch.js
+++ b/packages/website/components/w3up-launch.js
@@ -1,6 +1,6 @@
 import * as React from 'react';
 
-const sunsetStartEnv = process.env.NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_START ?? '2023-01-10T00:00:00Z';
+const sunsetStartEnv = process.env.NEXT_PUBLIC_W3UP_LAUNCH_SUNSET_START ?? '2024-01-10T00:00:00Z';
 const sunsetStartDate = new Date(Date.parse(sunsetStartEnv));
 
 /**
@@ -27,6 +27,15 @@ export const shouldShowSunsetAnnouncement = (at = new Date(), announcementStartD
   return announcementStartDate && at > announcementStartDate;
 };
 
+/**
+ * return whether the website should prevent allowing customers to switch their storage subscription plan.
+ * @param {Date} at - time at which to return whether to show the announcement
+ * @param {Date|undefined} [startDate] - when to begin preventing
+ */
+export const shouldPreventPlanSwitching = (at = new Date(), startDate = sunsetStartDate) => {
+  return at > startDate;
+};
+
 export const w3upLaunchConfig = {
   type: 'W3upLaunchConfig',
   stages: {
diff --git a/packages/website/modules/zero/components/tooltip/tooltip.js b/packages/website/modules/zero/components/tooltip/tooltip.js
index da41f0129c..f79a06bf2a 100644
--- a/packages/website/modules/zero/components/tooltip/tooltip.js
+++ b/packages/website/modules/zero/components/tooltip/tooltip.js
@@ -1,13 +1,13 @@
 import clsx from 'clsx';
 import { BsFillInfoCircleFill } from 'react-icons/bs';
 
-
 /**
  * @typedef {Object} InfoProps
  * @property {string} content
  * @property {string} [position]
  * @property {React.ReactNode} [icon]
  * @property {string|React.ReactNode} [children]
+ * @property {keyof JSX.IntrinsicElements} [element] - element name to use as root
  * @prop {string} [className]
  */
 
@@ -16,13 +16,14 @@ import { BsFillInfoCircleFill } from 'react-icons/bs';
  * @param {InfoProps} props
  * @returns
  */
-const Tooltip = ({ children, content, icon = null, className, position }) => {
+const Tooltip = ({ children, content, icon = null, className, position, element = 'div' }) => {
+  const Tag = element;
   return (
-    <div role="tooltip" className={clsx(className, 'Tooltip', position)}>
+    <Tag role="tooltip" className={clsx(className, 'Tooltip', position)}>
       {!children && (icon || <BsFillInfoCircleFill />)}
       {children}
       <span className="tooltip-content" dangerouslySetInnerHTML={{ __html: content }} />
-    </div>
+    </Tag>
   );
 };
 
diff --git a/packages/website/pages/account/payment.js b/packages/website/pages/account/payment.js
index babdf3ebec..6341c3a118 100644
--- a/packages/website/pages/account/payment.js
+++ b/packages/website/pages/account/payment.js
@@ -18,7 +18,11 @@ import { plans, freePlan } from '../../components/contexts/plansContext';
 import { userBillingSettings } from '../../lib/api';
 import GeneralPageData from '../../content/pages/general.json';
 import constants from '../../lib/constants.js';
-import { W3upMigrationRecommendationCopy, shouldShowSunsetAnnouncement } from '../../components/w3up-launch.js';
+import {
+  W3upMigrationRecommendationCopy,
+  shouldShowSunsetAnnouncement,
+  shouldPreventPlanSwitching,
+} from '../../components/w3up-launch.js';
 import * as PageBannerPortal from '../../components/page-banner/page-banner-portal.js';
 
 /**
@@ -155,6 +159,7 @@ const PaymentSettingsPage = props => {
                 currentPlan={currentPlan}
                 setPlanSelection={setPlanSelection}
                 setIsPaymentPlanModalOpen={setIsPaymentPlanModalOpen}
+                disablePlanSwitching={shouldPreventPlanSwitching()}
               />
             )}