From 3bc068123cede7efd5e6ae4d0088636a71e47f96 Mon Sep 17 00:00:00 2001 From: Simon Backx Date: Wed, 6 Dec 2023 17:36:18 +0100 Subject: [PATCH] Wired up checkbox to toggle tier visibility (#19260) fixes PROD-200 --- .../membership/tiers/TierDetailModal.tsx | 46 +++++++++++++++++-- 1 file changed, 42 insertions(+), 4 deletions(-) diff --git a/apps/admin-x-settings/src/components/settings/membership/tiers/TierDetailModal.tsx b/apps/admin-x-settings/src/components/settings/membership/tiers/TierDetailModal.tsx index 9abc1ea4db..7ce5f5ef14 100644 --- a/apps/admin-x-settings/src/components/settings/membership/tiers/TierDetailModal.tsx +++ b/apps/admin-x-settings/src/components/settings/membership/tiers/TierDetailModal.tsx @@ -8,7 +8,7 @@ import {ErrorMessages, useForm, useHandleError} from '@tryghost/admin-x-framewor import {RoutingModalProps, useRouting} from '@tryghost/admin-x-framework/routing'; import {Tier, useAddTier, useBrowseTiers, useEditTier} from '@tryghost/admin-x-framework/api/tiers'; import {currencies, currencySelectGroups, validateCurrencyAmount} from '../../../../utils/currency'; -import {getSettingValues} from '@tryghost/admin-x-framework/api/settings'; +import {getSettingValues, useEditSettings} from '@tryghost/admin-x-framework/api/settings'; import {toast} from 'react-hot-toast'; export type TierFormState = Partial> & { @@ -22,12 +22,14 @@ const TierDetailModalContent: React.FC<{tier?: Tier}> = ({tier}) => { const {updateRoute} = useRouting(); const {mutateAsync: updateTier} = useEditTier(); const {mutateAsync: createTier} = useAddTier(); + const {mutateAsync: editSettings} = useEditSettings(); const [hasFreeTrial, setHasFreeTrial] = React.useState(!!tier?.trial_days); const handleError = useHandleError(); const {localSettings, siteData} = useSettingGroup(); - const siteTitle = getSettingValues(localSettings, ['title']) as string[]; + const [siteTitle, portalPlansJson] = getSettingValues(localSettings, ['title', 'portal_plans']) as string[]; const hasPortalImprovements = useFeatureFlag('portalImprovements'); const allowNameChange = !isFreeTier || hasPortalImprovements; + const portalPlans = JSON.parse(portalPlansJson?.toString() || '[]') as string[]; const validators: {[key in keyof Tier]?: () => string | undefined} = { name: () => (formState.name ? undefined : 'You must specify a name'), @@ -70,6 +72,31 @@ const TierDetailModalContent: React.FC<{tier?: Tier}> = ({tier}) => { } else { await createTier(values); } + if (isFreeTier && hasPortalImprovements) { + // If we changed the visibility, we also need to update Portal settings in some situations + // Like the free tier is a special case, and should also be present/absent in portal_plans + const visible = formState.visibility === 'public'; + let save = false; + + if (portalPlans.includes('free') && !visible) { + portalPlans.splice(portalPlans.indexOf('free'), 1); + save = true; + } + + if (!portalPlans.includes('free') && visible) { + portalPlans.push('free'); + save = true; + } + + if (save) { + await editSettings([ + { + key: 'portal_plans', + value: JSON.stringify(portalPlans) + } + ]); + } + } }, onSavedStateReset: () => { modal.remove(); @@ -335,8 +362,19 @@ const TierDetailModalContent: React.FC<{tier?: Tier}> = ({tier}) => {
- - {!tier && hasPortalImprovements &&
{}}>
} + + {hasPortalImprovements && +
+ { + updateForm(state => ({...state, visibility: checked ? 'public' : 'none'})); + }} + > +
+ }
;