Merge pull request #1417 from guardian/pf/switch-for-iconik-ui

Add cookie-based switch for whether or not to show Iconik UI

Author: bryophyta

app/controllers/IconikController.scala

@@ -6,7 +6,13 @@ import com.gu.pandahmac.HMACAuthActions
 import com.typesafe.config.Config
 import data.{DataStores, UnpackedDataStores}
 import play.api.libs.json.Json
-import play.api.mvc.{Action, AnyContent, BaseController, ControllerComponents}
+import play.api.mvc.{
+  Action,
+  AnyContent,
+  BaseController,
+  ControllerComponents,
+  Cookie
+}
 import util.AWSConfig
 
 class IconikController(
@@ -20,7 +26,15 @@ class IconikController(
     with JsonRequestParsing
     with Logging {
 
-  import authActions.{APIAuthAction, APIHMACAuthAction}
+  import authActions.{APIAuthAction, APIHMACAuthAction, AuthAction}
+
+  def switchShowIconikUiOn(): Action[AnyContent] = AuthAction { implicit req =>
+    Redirect("/", FOUND).withCookies(Cookie("showIconik", "true"))
+  }
+
+  def switchShowIconikUiOff(): Action[AnyContent] = AuthAction { implicit req =>
+    Redirect("/", FOUND).withCookies(Cookie("showIconik", "false"))
+  }
 
   def getWorkingGroup(groupId: String): Action[AnyContent] = APIAuthAction {
     stores.iconikDataStore.getWorkingGroup(groupId) match {

app/controllers/VideoUIApp.scala

@@ -8,7 +8,13 @@ import com.gu.pandomainauth.model.User
 import model.{ClientConfig, Presence}
 import play.api.Configuration
 import play.api.libs.json.Json
-import play.api.mvc.{Action, AnyContent, BaseController, ControllerComponents}
+import play.api.mvc.{
+  Action,
+  AnyContent,
+  BaseController,
+  ControllerComponents,
+  Cookie
+}
 import util.{AWSConfig, TrainingMode}
 import views.html.helper.CSRF
 
@@ -30,6 +36,11 @@ class VideoUIApp(
 
   def index(id: String = ""): Action[AnyContent] = AuthAction { implicit req =>
     val isTrainingMode = isInTrainingMode(req)
+    val shouldShowIconikUi = req.cookies.get("showIconik") match {
+      case Some(cookie) if cookie.value == "true"  => true
+      case Some(cookie) if cookie.value == "false" => false
+      case None => conf.get[String]("stage") != "PROD"
+    }
 
     val jsFileName = "video-ui/build/app.js"
 
@@ -61,7 +72,7 @@ class VideoUIApp(
       workflowUrl = awsConfig.workflowUrl,
       targetingUrl = awsConfig.targetingUrl,
       tagManagerUrl = awsConfig.tagManagerUrl,
-      showIconik = conf.get[String]("stage") != "PROD"
+      showIconik = shouldShowIconikUi
     )
 
     Ok(

conf/routes

@@ -80,3 +80,7 @@ GET            /healthcheck                                     controllers.Heal
 
 #static assets
 GET            /assets/*file                                    controllers.Assets.versioned(path="/public", file: Asset)
+
+
+GET           /switches/iconik/off                              controllers.IconikController.switchShowIconikUiOff()
+GET           /switches/iconik/on                               controllers.IconikController.switchShowIconikUiOn()