Commit f69d8170 authored by Sean McGivern's avatar Sean McGivern

Add feature categories to controllers beginning with I

parent 780f8065
...@@ -11,6 +11,8 @@ class IdeController < ApplicationController ...@@ -11,6 +11,8 @@ class IdeController < ApplicationController
push_frontend_feature_flag(:schema_linting) push_frontend_feature_flag(:schema_linting)
end end
feature_category :web_ide
def index def index
Gitlab::UsageDataCounters::WebIdeCounter.increment_views_count Gitlab::UsageDataCounters::WebIdeCounter.increment_views_count
end end
......
# frozen_string_literal: true # frozen_string_literal: true
class Import::AvailableNamespacesController < ApplicationController class Import::AvailableNamespacesController < ApplicationController
feature_category :importers
def index def index
render json: NamespaceSerializer.new.represent(current_user.manageable_groups_with_routes) render json: NamespaceSerializer.new.represent(current_user.manageable_groups_with_routes)
end end
......
...@@ -4,6 +4,7 @@ class Import::BaseController < ApplicationController ...@@ -4,6 +4,7 @@ class Import::BaseController < ApplicationController
include ActionView::Helpers::SanitizeHelper include ActionView::Helpers::SanitizeHelper
before_action :import_rate_limit, only: [:create] before_action :import_rate_limit, only: [:create]
feature_category :importers
def status def status
respond_to do |format| respond_to do |format|
......
...@@ -4,6 +4,8 @@ class Import::BulkImportsController < ApplicationController ...@@ -4,6 +4,8 @@ class Import::BulkImportsController < ApplicationController
before_action :ensure_group_import_enabled before_action :ensure_group_import_enabled
before_action :verify_blocked_uri, only: :status before_action :verify_blocked_uri, only: :status
feature_category :importers
def configure def configure
session[access_token_key] = params[access_token_key]&.strip session[access_token_key] = params[access_token_key]&.strip
session[url_key] = params[url_key] session[url_key] = params[url_key]
......
...@@ -6,6 +6,8 @@ class Import::GitlabGroupsController < ApplicationController ...@@ -6,6 +6,8 @@ class Import::GitlabGroupsController < ApplicationController
before_action :ensure_group_import_enabled before_action :ensure_group_import_enabled
before_action :import_rate_limit, only: %i[create] before_action :import_rate_limit, only: %i[create]
feature_category :importers
def create def create
unless file_is_valid?(group_params[:file]) unless file_is_valid?(group_params[:file])
return redirect_back_or_default(options: { alert: s_('GroupImport|Unable to process group import file') }) return redirect_back_or_default(options: { alert: s_('GroupImport|Unable to process group import file') })
......
...@@ -12,6 +12,8 @@ class InvitesController < ApplicationController ...@@ -12,6 +12,8 @@ class InvitesController < ApplicationController
respond_to :html respond_to :html
feature_category :authentication_and_authorization
def show def show
track_new_user_invite_experiment('opened') track_new_user_invite_experiment('opened')
accept if skip_invitation_prompt? accept if skip_invitation_prompt?
......
...@@ -26,7 +26,7 @@ RSpec.describe "Every controller" do ...@@ -26,7 +26,7 @@ RSpec.describe "Every controller" do
next if controller.feature_category_for_action(action) next if controller.feature_category_for_action(action)
next unless controller.to_s.start_with?('B', 'C', 'D', 'E', 'F', next unless controller.to_s.start_with?('B', 'C', 'D', 'E', 'F',
'H', 'H', 'I',
'Projects::MergeRequestsController') 'Projects::MergeRequestsController')
"#{controller}##{action}" "#{controller}##{action}"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment