github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-21 20:47:08 -05:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #17277 from overleaf/jpa-hide-old-writefull-banner

Browse source 
[web] hide old Writefull banner when the extension is already installed

GitOrigin-RevId: 35a1a3f6e8dd5860d81ea61ea01c40419454d40c
This commit is contained in:
Jimmy Domagala-Tang 2024-02-22 11:48:43 -08:00 committed by Copybot
parent f4454cfe7e
commit 404e023e08
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
services/web/frontend/js/features/project-list/components/notifications/user-notifications.tsx
View file

@ -58,9 +58,13 @@ function UserNotifications() {
return false return false
} }
const hasWritefullExtensionAlreadyInstalled =
window.writefull?.type === 'extension'
const show = const show =
user?.writefull?.enabled === true || // show to any users who have writefull enabled regardless of split test 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 (!writefullIntegrationSplitTestEnabled && // show old banner to users who are not in the split test, who are on chrome and havent dismissed
!hasWritefullExtensionAlreadyInstalled &&
isChromium() && isChromium() &&
getMeta('ol-showWritefullPromoBanner')) getMeta('ol-showWritefullPromoBanner'))