Add feature flag

Author: mrmir

config/initializers/feature_decisions.rb

@@ -64,3 +64,8 @@
 OpenProject::FeatureDecisions.add :comments_with_restricted_visibility,
                                   description: "Enables submitting comments that are restricted " \
                                                "and only a subset of users can see"
+
+OpenProject::FeatureDecisions.add :meeting_backlogs,
+                                  description: "Enables backlogs for meetings. " \
+                                               "Note: Meetings created with this flag off will not be accessible " \
+                                               "after activating it."

modules/meeting/app/components/meeting_agenda_items/item_component/show_component.html.erb

@@ -77,9 +77,9 @@
               move_actions(menu)
             end
 
-            menu.with_divider
-            move_to_current_meeting_action_item(menu) if in_backlog?
-            move_to_backlog_action_item(menu) if !in_backlog?
+            menu.with_divider if OpenProject::FeatureDecisions.meeting_backlogs_active?
+            move_to_current_meeting_action_item(menu) if in_backlog? && OpenProject::FeatureDecisions.meeting_backlogs_active?
+            move_to_backlog_action_item(menu) if !in_backlog? && OpenProject::FeatureDecisions.meeting_backlogs_active?
 
             if move_to_next_meeting_enabled?
               menu.with_divider

modules/meeting/app/components/meeting_agenda_items/item_component/show_component.rb

@@ -63,7 +63,7 @@ def add_outcome_action?
       @meeting_agenda_item.editable? &&
         @meeting.in_progress? &&
         !@meeting_agenda_item.outcomes.exists? &&
-        @meeting_agenda_item.in_backlog?
+        (!OpenProject::FeatureDecisions.meeting_backlogs_active? || @meeting_agenda_item.in_backlog?)
     end
 
     def first?

modules/meeting/app/components/meeting_agenda_items/outcomes/new_button_component.rb

@@ -48,7 +48,7 @@ def wrapper_uniq_by
     def render?
       @meeting.in_progress? &&
         User.current.allowed_in_project?(:create_meeting_minutes, @meeting.project) &&
-        @meeting_agenda_item.in_backlog?
+        (!OpenProject::FeatureDecisions.meeting_backlogs_active? || @meeting_agenda_item.in_backlog?)
     end
   end
 end

modules/meeting/app/components/meeting_agenda_items/outcomes/show_notes_component.rb

@@ -47,7 +47,7 @@ def initialize(meeting_outcome:)
     def edit_enabled?
       @meeting.in_progress? &&
         User.current.allowed_in_project?(:create_meeting_minutes, @meeting.project) &&
-        @agenda_item.in_backlog?
+        (!OpenProject::FeatureDecisions.meeting_backlogs_active? || @agenda_item.in_backlog?)
     end
 
     def edit_action_item(menu)

modules/meeting/app/components/meeting_sections/show_component.html.erb

@@ -5,7 +5,7 @@
         component.with_header(font_weight: :bold, pl: 0) do
           render(MeetingSections::HeaderComponent.new(meeting_section: @meeting_section, state: @state))
         end
-      elsif render_backlog? && !@meeting.closed?
+      elsif OpenProject::FeatureDecisions.meeting_backlogs_active? && render_backlog? && !@meeting.closed?
         component.with_header do
           render(MeetingSections::Backlogs::HeaderComponent.new(backlog: @meeting_section, box: component))
         end

modules/meeting/app/components/meetings/show_component.html.erb

@@ -16,8 +16,10 @@
               render(MeetingAgendaItems::NewButtonComponent.new(meeting: @meeting))
             end
 
-            agenda.with_row do
-              render(MeetingSections::Backlogs::ContainerComponent.new(meeting: @meeting))
+            if OpenProject::FeatureDecisions.meeting_backlogs_active?
+              agenda.with_row do
+                render(MeetingSections::Backlogs::ContainerComponent.new(meeting: @meeting))
+              end
             end
 
             agenda.with_row(mt: 3, display: [:block, nil, :none]) do

modules/meeting/app/contracts/meeting_outcomes/editable_item.rb

@@ -33,7 +33,8 @@ module EditableItem
     extend ActiveSupport::Concern
 
     included do
-      validate :validate_editable, :validate_meeting_existence, :user_allowed_to_add, :validate_not_in_backlog
+      validate :validate_editable, :validate_meeting_existence, :user_allowed_to_add
+      validate :validate_not_in_backlog if OpenProject::FeatureDecisions.meeting_backlogs_active?
     end
 
     protected

modules/meeting/app/controllers/concerns/meetings/agenda_component_streams.rb

@@ -143,7 +143,8 @@ def update_new_button_via_turbo_stream(disabled: false, meeting: @meeting, meeti
       end
 
       def render_agenda_item_form_via_turbo_stream(meeting: @meeting, meeting_section: @meeting_section, type: :simple)
-        if meeting.sections.empty? && meeting_section != meeting.backlog
+        if meeting.sections.empty? &&
+          (!OpenProject::FeatureDecisions.meeting_backlogs_active? || meeting_section != meeting.backlog)
           render_agenda_item_form_for_empty_meeting_via_turbo_stream(type:)
         else
           render_agenda_item_form_in_section_via_turbo_stream(meeting:, meeting_section:, type:)
@@ -216,7 +217,8 @@ def update_item_via_turbo_stream(state: :show, meeting_agenda_item: @meeting_age
       def add_item_via_turbo_stream(meeting_agenda_item: @meeting_agenda_item, clear_slate: false) # rubocop:disable Metrics/AbcSize
         if clear_slate
           update_list_via_turbo_stream(form_hidden: false, form_type: @agenda_item_type)
-        elsif meeting_agenda_item.meeting.agenda_items.count == 1 && meeting_agenda_item.meeting.sections.present?
+        elsif meeting_agenda_item.meeting.agenda_items.count == 1 &&
+          (!OpenProject::FeatureDecisions.meeting_backlogs_active? || meeting_agenda_item.meeting.sections.present?)
           update_list_via_turbo_stream(form_hidden: true)
 
           update_new_component_via_turbo_stream(
@@ -349,11 +351,13 @@ def update_section_header_via_turbo_stream(meeting_section: @meeting_section, st
       end
 
       def update_backlog_via_turbo_stream(meeting: @meeting)
-        update_via_turbo_stream(
-          component: MeetingSections::Backlogs::ContainerComponent.new(
-            meeting: meeting
+        if OpenProject::FeatureDecisions.meeting_backlogs_active?
+          update_via_turbo_stream(
+            component: MeetingSections::Backlogs::ContainerComponent.new(
+              meeting: meeting
+            )
           )
-        )
+        end
       end
 
       def update_section_via_turbo_stream(meeting_section: @meeting_section, form_hidden: true, form_type: :simple,

modules/meeting/app/controllers/meeting_outcomes_controller.rb

@@ -41,7 +41,7 @@ class MeetingOutcomesController < ApplicationController
   def new
     update_all_via_turbo_stream
 
-    if @meeting.in_progress? && @meeting_agenda_item.in_backlog?
+    if @meeting.in_progress? && (!OpenProject::FeatureDecisions.meeting_backlogs_active? || @meeting_agenda_item.in_backlog?)
       render_base_outcome_component_via_turbo_stream(meeting: @meeting, meeting_agenda_item: @meeting_agenda_item,
                                                      meeting_outcome: nil, edit: true)
     else

modules/meeting/app/controllers/work_package_meetings_tab_controller.rb

@@ -102,6 +102,8 @@ def add_work_package_to_meeting_params
   end
 
   def backlog_id
+    return unless OpenProject::FeatureDecisions.meeting_backlogs_active?
+
     meeting_id = add_work_package_to_meeting_params[:meeting_id]
     return if meeting_id.blank?
 

modules/meeting/app/services/meetings/create_service.rb

@@ -43,7 +43,7 @@ def after_perform(call)
         end
       end
 
-      if call.success?
+      if call.success? && OpenProject::FeatureDecisions.meeting_backlogs_active?
         backlog = create_backlog(call.result)
         call.merge!(backlog)
       end