diff --git a/services/web/app/src/Features/Project/ProjectListController.js b/services/web/app/src/Features/Project/ProjectListController.js index 249dd6c333..99ae52984a 100644 --- a/services/web/app/src/Features/Project/ProjectListController.js +++ b/services/web/app/src/Features/Project/ProjectListController.js @@ -360,7 +360,7 @@ async function projectListPage(req, res, next) { await SplitTestHandler.promises.getAssignment( req, res, - 'writefull-integration' + 'writefull-oauth-promotion' ) let showInrGeoBanner = false diff --git a/services/web/app/src/Features/User/UserPagesController.js b/services/web/app/src/Features/User/UserPagesController.js index dab34a619b..b1af6d5546 100644 --- a/services/web/app/src/Features/User/UserPagesController.js +++ b/services/web/app/src/Features/User/UserPagesController.js @@ -86,7 +86,7 @@ async function settingsPage(req, res) { await SplitTestHandler.promises.getAssignment( req, res, - 'writefull-integration' + 'writefull-oauth-promotion' ) let personalAccessTokens 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 586aae6093..4d02a01faf 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 @@ -45,8 +45,8 @@ function UserNotifications() { ) const showInrGeoBanner = getMeta('ol-showInrGeoBanner', false) const showBrlGeoBanner = getMeta('ol-showBrlGeoBanner', false) - const writefullIntegrationSplitTestEnabled = isSplitTestEnabled( - 'writefull-integration' + const writefullOauthPromotionSplitTestEnabled = isSplitTestEnabled( + 'writefull-oauth-promotion' ) const user = getMeta('ol-user') @@ -61,7 +61,7 @@ function UserNotifications() { 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 + (!writefullOauthPromotionSplitTestEnabled && // show old banner to users who are not in the split test, who are on chrome and havent dismissed isChromium() && getMeta('ol-showWritefullPromoBanner')) @@ -71,7 +71,7 @@ function UserNotifications() { page: '/project', name: user?.writefull?.enabled === true || - writefullIntegrationSplitTestEnabled + writefullOauthPromotionSplitTestEnabled ? 'writefull-premium' : 'writefull', }) @@ -84,7 +84,7 @@ function UserNotifications() { const hasWritefullExtensionAlreadyInstalled = window.writefull?.type === 'extension' const usesWritefullIntegration = - writefullIntegrationSplitTestEnabled || user?.writefull?.enabled + writefullOauthPromotionSplitTestEnabled || user?.writefull?.enabled const writefullBannerVariant = hasWritefullExtensionAlreadyInstalled || usesWritefullIntegration ? 'plans-page' diff --git a/services/web/frontend/js/features/settings/components/linking-section.tsx b/services/web/frontend/js/features/settings/components/linking-section.tsx index bb50afdc71..e12d49b881 100644 --- a/services/web/frontend/js/features/settings/components/linking-section.tsx +++ b/services/web/frontend/js/features/settings/components/linking-section.tsx @@ -59,7 +59,7 @@ function LinkingSection() { // even if they arent in the split test, if they have it enabled let them toggle it off const user = getMeta('ol-user') const shouldLoadWritefull = - (splitTestVariants['writefull-integration'] === 'enabled' || + (splitTestVariants['writefull-oauth-promotion'] === 'enabled' || user.writefull?.enabled === true) && !window.writefull // check if the writefull extension is installed, in which case we dont handle the integration diff --git a/services/web/test/frontend/features/project-list/components/notifications.test.tsx b/services/web/test/frontend/features/project-list/components/notifications.test.tsx index 9979c7fe8b..b88f220b1e 100644 --- a/services/web/test/frontend/features/project-list/components/notifications.test.tsx +++ b/services/web/test/frontend/features/project-list/components/notifications.test.tsx @@ -977,10 +977,10 @@ describe('', function () { window.metaAttributesCache = window.metaAttributesCache || new Map() }) - describe('when writefull-integration split test is not enabled', function () { + describe('when writefull-oauth-promotion split test is not enabled', function () { beforeEach(function () { window.metaAttributesCache.set('ol-splitTestVariants', { - 'writefull-integration': 'default', + 'writefull-oauth-promotion': 'default', }) window.metaAttributesCache.set('ol-user', { writefull: { enabled: false }, @@ -1017,10 +1017,10 @@ describe('', function () { }) }) - describe('when writefull-integration split test is enabled', function () { + describe('when writefull-oauth-promotion split test is enabled', function () { beforeEach(function () { window.metaAttributesCache.set('ol-splitTestVariants', { - 'writefull-integration': 'enabled', + 'writefull-oauth-promotion': 'enabled', }) })