Update to latest n-desktop-app-banner and remove subscriberCohort che…

…ck for it 🐿 v2.5.16
Financial-Times · Nov 9, 2017 · 2f4fd8a · 2f4fd8a
1 parent cbe5877
commit 2f4fd8a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/bower.json b/bower.json
@@ -34,7 +34,7 @@
     "next-session-client": "^2.3.1",
     "n-counter-ad-blocking": "^3.1.2",
     "n-topic-search": "^1.0.0",
-    "n-desktop-app-banner": "^1.0.6"
+    "n-desktop-app-banner": "^2.0.0"
   },
   "resolutions": {
     "n-ui-foundations": "^2.4.3"

diff --git a/server/index.js b/server/index.js
@@ -73,7 +73,7 @@ module.exports = options => {
 	app.use(function (req, res, next) {
 		app.locals.__disableAndroidBanner = (!res.locals.flags.subscriberCohort || res.locals.flags.disableAndroidSmartBanner);
 		app.locals.__disableIosSmartBanner = (!res.locals.flags.subscriberCohort || res.locals.flags.disableIosSmartBanner);
-		app.locals.__enableDesktopAppBanner = res.locals.flags.subscriberCohort && res.locals.flags.onboardingMessaging === 'appPromotingBanner';
+		app.locals.__enableDesktopAppBanner = res.locals.flags.onboardingMessaging === 'appPromotingBanner';
 
 		next();
 	});