From 3e1b090ad210b429fbf02ce53f9d37d92b006b82 Mon Sep 17 00:00:00 2001 From: Jimmy Domagala-Tang Date: Thu, 22 Feb 2024 12:31:29 -0800 Subject: [PATCH] Merge pull request #17279 from overleaf/revert-17277-jpa-hide-old-writefull-banner Revert "[web] hide old Writefull banner when the extension is already installed" GitOrigin-RevId: bc4d28cff7ce472a5f271026122c5fc6c4ab7886 --- .../components/notifications/user-notifications.tsx | 4 ---- 1 file changed, 4 deletions(-) diff --git a/services/web/frontend/js/features/project-list/components/notifications/user-notifications.tsx b/services/web/frontend/js/features/project-list/components/notifications/user-notifications.tsx index 39995ab7b1..58b64cb199 100644 --- a/services/web/frontend/js/features/project-list/components/notifications/user-notifications.tsx +++ b/services/web/frontend/js/features/project-list/components/notifications/user-notifications.tsx @@ -58,13 +58,9 @@ function UserNotifications() { return false } - const hasWritefullExtensionAlreadyInstalled = - window.writefull?.type === 'extension' - const show = user?.writefull?.enabled === true || // show to any users who have writefull enabled regardless of split test (!writefullIntegrationSplitTestEnabled && // show old banner to users who are not in the split test, who are on chrome and havent dismissed - !hasWritefullExtensionAlreadyInstalled && isChromium() && getMeta('ol-showWritefullPromoBanner'))