From aa61f8dfbbccc9fecb7cee20af1cd2feccffcf48 Mon Sep 17 00:00:00 2001
From: Agniva De Sarker <agnivade@yahoo.co.in>
Date: Mon, 30 Jan 2023 22:23:20 +0530
Subject: [PATCH] Revert "Set BoardsProduct FF to on by default" (#22177)

This reverts commit 6b6150d37453403eaf834904bc4139aa2d8d1bed.
---
 config/diff_test.go    | 6 +++---
 model/config.go        | 4 ++--
 model/config_test.go   | 4 ++--
 model/feature_flags.go | 2 +-
 scripts/test.sh        | 1 -
 5 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/config/diff_test.go b/config/diff_test.go
index 427045dae7..15ec6f41f7 100644
--- a/config/diff_test.go
+++ b/config/diff_test.go
@@ -808,7 +808,7 @@ func TestDiff(t *testing.T) {
 							Enable: !defaultConfigGen().PluginSettings.PluginStates["com.mattermost.nps"].Enable,
 						},
 						"focalboard": {
-							Enable: false,
+							Enable: true,
 						},
 						"playbooks": {
 							Enable: true,
@@ -846,7 +846,7 @@ func TestDiff(t *testing.T) {
 							Enable: true,
 						},
 						"focalboard": {
-							Enable: false,
+							Enable: true,
 						},
 						"playbooks": {
 							Enable: true,
@@ -876,7 +876,7 @@ func TestDiff(t *testing.T) {
 					BaseVal: defaultConfigGen().PluginSettings.PluginStates,
 					ActualVal: map[string]*model.PluginState{
 						"focalboard": {
-							Enable: false,
+							Enable: true,
 						},
 						"playbooks": {
 							Enable: true,
diff --git a/model/config.go b/model/config.go
index ef948375cf..ad169fcbbf 100644
--- a/model/config.go
+++ b/model/config.go
@@ -2867,8 +2867,8 @@ func (s *PluginSettings) SetDefaults(ls LogSettings) {
 	}
 
 	if s.PluginStates[PluginIdFocalboard] == nil {
-		// Disable the focalboard plugin by default
-		s.PluginStates[PluginIdFocalboard] = &PluginState{Enable: false}
+		// Enable the focalboard plugin by default
+		s.PluginStates[PluginIdFocalboard] = &PluginState{Enable: true}
 	}
 
 	if s.PluginStates[PluginIdApps] == nil {
diff --git a/model/config_test.go b/model/config_test.go
index 752ace1547..cedffbcb08 100644
--- a/model/config_test.go
+++ b/model/config_test.go
@@ -387,11 +387,11 @@ func TestConfigDefaultChannelExportPluginState(t *testing.T) {
 }
 
 func TestConfigDefaultFocalboardPluginState(t *testing.T) {
-	t.Run("should not enable Focalboard plugin by default", func(t *testing.T) {
+	t.Run("should enable Focalboard plugin by default", func(t *testing.T) {
 		c1 := Config{}
 		c1.SetDefaults()
 
-		assert.False(t, c1.PluginSettings.PluginStates["focalboard"].Enable)
+		assert.True(t, c1.PluginSettings.PluginStates["focalboard"].Enable)
 	})
 
 	t.Run("should not re-enable focalboard plugin after it has been disabled", func(t *testing.T) {
diff --git a/model/feature_flags.go b/model/feature_flags.go
index 449a04b5bc..4864a660c1 100644
--- a/model/feature_flags.go
+++ b/model/feature_flags.go
@@ -95,7 +95,7 @@ func (f *FeatureFlags) SetDefaults() {
 	f.InsightsEnabled = true
 	f.CommandPalette = false
 	f.CallsEnabled = true
-	f.BoardsProduct = true
+	f.BoardsProduct = false
 	f.SendWelcomePost = true
 	f.PostPriority = true
 	f.PeopleProduct = false
diff --git a/scripts/test.sh b/scripts/test.sh
index f13d9244c5..63f89fc785 100755
--- a/scripts/test.sh
+++ b/scripts/test.sh
@@ -14,7 +14,6 @@ COVERMODE=$8
 
 PACKAGES_COMMA=$(echo $PACKAGES | tr ' ' ',')
 export MM_SERVER_PATH=$PWD
-export MM_FEATUREFLAGS_BoardsProduct=false
 
 echo "Packages to test: $PACKAGES"
 echo "GOFLAGS: $GOFLAGS"