Merge branch 'main' into al-update-cmp-13-8-0

common/app/conf/switches/JournalismSwitches.scala

@@ -62,4 +62,14 @@ trait JournalismSwitches {
     sellByDate = never,
     exposeClientSide = false,
   )
+
+  val Lightbox = Switch(
+    SwitchGroup.Journalism,
+    name = "lightbox",
+    description = "Enable lightbox for images",
+    owners = Seq(Owner.withEmail("[email protected]")),
+    safeState = On,
+    sellByDate = never,
+    exposeClientSide = true,
+  )
 }

common/app/experiments/Experiments.scala

@@ -11,7 +11,6 @@ import java.time.LocalDate
 object ActiveExperiments extends ExperimentsDefinition {
   override val allExperiments: Set[Experiment] =
     Set(
-      Lightbox,
       AdaptiveSite,
       DeeplyRead,
     )
@@ -27,15 +26,6 @@ object DeeplyRead
       participationGroup = Perc50,
     )
 
-object Lightbox
-    extends Experiment(
-      name = "lightbox",
-      description = "Testing the impact lightbox might have on our CWVs",
-      owners = Seq(Owner.withEmail("[email protected]")),
-      sellByDate = LocalDate.of(2024, 1, 31),
-      participationGroup = Perc5A,
-    )
-
 // Removing while we are still implementing this content type in DCR
 //object DCRImageContent
 //    extends Experiment(