From 56235da7dd568d766c7afee102a111700943c5b1 Mon Sep 17 00:00:00 2001 From: Miguel Serrano Date: Mon, 11 Aug 2025 11:29:36 +0200 Subject: [PATCH] [web] Clean up `group-audit-logs` split test (#27614) GitOrigin-RevId: e22183820e418cabc83cf2619c2c10cf4f82d3b9 --- .../Subscription/SubscriptionController.js | 4 ---- .../dashboard/managed-group-subscriptions.tsx | 18 ++++++------------ .../managed-group-subscriptions.test.tsx | 4 ---- 3 files changed, 6 insertions(+), 20 deletions(-) diff --git a/services/web/app/src/Features/Subscription/SubscriptionController.js b/services/web/app/src/Features/Subscription/SubscriptionController.js index 6c2ad53a92..def0abf7ac 100644 --- a/services/web/app/src/Features/Subscription/SubscriptionController.js +++ b/services/web/app/src/Features/Subscription/SubscriptionController.js @@ -177,10 +177,6 @@ async function userSubscriptionPage(req, res) { premiumSource: aiAssistViaWritefullSource, } = await UserGetter.promises.getWritefullData(user._id) - // populate splitTestVariants in locals with a value for the split test, so - // it's available in meta ol-splitTestVariants in the browser - await SplitTestHandler.promises.getAssignment(req, res, 'group-audit-logs') - const data = { title: 'your_subscription', plans: plansData?.plans, diff --git a/services/web/frontend/js/features/subscription/components/dashboard/managed-group-subscriptions.tsx b/services/web/frontend/js/features/subscription/components/dashboard/managed-group-subscriptions.tsx index 0ad9274af6..9af548ceb1 100644 --- a/services/web/frontend/js/features/subscription/components/dashboard/managed-group-subscriptions.tsx +++ b/services/web/frontend/js/features/subscription/components/dashboard/managed-group-subscriptions.tsx @@ -97,10 +97,6 @@ export default function ManagedGroupSubscriptions() { getMeta('ol-groupSettingsAdvertisedFor') || [] const groupSettingsEnabledFor = getMeta('ol-groupSettingsEnabledFor') || [] - const splitTestVariants = getMeta('ol-splitTestVariants') - const displayAuditLogsLink = - splitTestVariants?.['group-audit-logs'] === 'active' - return ( <> {managedGroupSubscriptions.map(subscription => { @@ -129,14 +125,12 @@ export default function ManagedGroupSubscriptions() { {groupSettingsAdvertisedFor?.includes(subscription._id) && ( )} - {displayAuditLogsLink && ( - - )} + ', function () { value: managedGroupSubscriptions, }, { name: 'ol-usersEmail', value: 'admin@example.com' }, - { - name: 'ol-splitTestVariants', - value: { 'group-audit-logs': 'active' }, - }, ], })