From daec240f8d7c3410722de542945833a8b661b4f9 Mon Sep 17 00:00:00 2001 From: Ian Bicking Date: Mon, 7 Jan 2019 15:20:34 -0600 Subject: [PATCH] Fix #5254 and fix #5253, remove FxA marketing (#5272) This simply says to never show the promotions (but does not remove the promotion code). --- server/src/promotion-strategy.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/server/src/promotion-strategy.js b/server/src/promotion-strategy.js index 6de635a4d9..139739139a 100644 --- a/server/src/promotion-strategy.js +++ b/server/src/promotion-strategy.js @@ -1,6 +1,9 @@ class PromotionStrategy { /* Display Firefox Account onboarding dialog in header below signin button */ shouldShowOnboardingDialog(isOwner, isShotPage) { + // Because we are deprecating the server, we never want to show this. + return false; + /* let show = false; // Exit without showing dialog on non-owner shot page if (isShotPage && !isOwner) { @@ -16,6 +19,7 @@ class PromotionStrategy { show = true; } return show; + */ } /* Display edit tool promotion on shot page below edit button */ @@ -59,10 +63,14 @@ class PromotionStrategy { /* Display FxA signin ad-banner inside header */ shouldShowFxaBanner(hasFxa) { + // Because we are deprecating the server, we never want to show this. + return false; + /* if (!hasFxa) { return true; } return false; + */ } shouldShowDeprecation() {