Commit dd77a894 authored by Sean McGivern's avatar Sean McGivern

Add feature categories to ProjectsController::[I-L]*

Danger wants a commit body, but there's really not any more to say.
These are all best effort and not guaranteed to be 100% accurate.
parent c11ebd30
...@@ -7,6 +7,8 @@ module Projects ...@@ -7,6 +7,8 @@ module Projects
before_action :authorize_read_project! before_action :authorize_read_project!
before_action :validate_jira_import_settings! before_action :validate_jira_import_settings!
feature_category :integrations
def show def show
end end
......
...@@ -11,6 +11,8 @@ class Projects::ImportsController < Projects::ApplicationController ...@@ -11,6 +11,8 @@ class Projects::ImportsController < Projects::ApplicationController
before_action :redirect_if_progress, except: :show before_action :redirect_if_progress, except: :show
before_action :redirect_if_no_import, only: :show before_action :redirect_if_no_import, only: :show
feature_category :importers
def new def new
end end
......
...@@ -10,6 +10,8 @@ module Projects ...@@ -10,6 +10,8 @@ module Projects
prepend_before_action :project_without_auth prepend_before_action :project_without_auth
feature_category :incident_management
def create def create
result = webhook_processor.execute(params[:token]) result = webhook_processor.execute(params[:token])
......
...@@ -7,6 +7,8 @@ class Projects::IncidentsController < Projects::ApplicationController ...@@ -7,6 +7,8 @@ class Projects::IncidentsController < Projects::ApplicationController
before_action :authorize_read_issue! before_action :authorize_read_issue!
before_action :load_incident, only: [:show] before_action :load_incident, only: [:show]
feature_category :incident_management
def index def index
end end
......
...@@ -7,6 +7,8 @@ module Projects ...@@ -7,6 +7,8 @@ module Projects
before_action :authorize_admin_issue_link!, only: [:create, :destroy] before_action :authorize_admin_issue_link!, only: [:create, :destroy]
before_action :authorize_issue_link_association!, only: :destroy before_action :authorize_issue_link_association!, only: :destroy
feature_category :issue_tracking
private private
def authorize_admin_issue_link! def authorize_admin_issue_link!
......
...@@ -65,6 +65,17 @@ class Projects::IssuesController < Projects::ApplicationController ...@@ -65,6 +65,17 @@ class Projects::IssuesController < Projects::ApplicationController
alias_method :designs, :show alias_method :designs, :show
feature_category :issue_tracking, [
:index, :calendar, :show, :new, :create, :edit, :update,
:destroy, :move, :reorder, :designs, :toggle_subscription,
:discussions, :bulk_update, :realtime_changes,
:toggle_award_emoji, :mark_as_spam, :related_branches,
:can_create_branch, :create_merge_request
]
feature_category :service_desk, [:service_desk]
feature_category :importers, [:import_csv, :export_csv]
def index def index
@issues = @issuables @issues = @issuables
......
...@@ -16,6 +16,8 @@ class Projects::JobsController < Projects::ApplicationController ...@@ -16,6 +16,8 @@ class Projects::JobsController < Projects::ApplicationController
layout 'project' layout 'project'
feature_category :continuous_integration
def index def index
# We need all builds for tabs counters # We need all builds for tabs counters
@all_builds = Ci::JobsFinder.new(current_user: current_user, project: @project).execute @all_builds = Ci::JobsFinder.new(current_user: current_user, project: @project).execute
......
...@@ -15,6 +15,8 @@ class Projects::LabelsController < Projects::ApplicationController ...@@ -15,6 +15,8 @@ class Projects::LabelsController < Projects::ApplicationController
respond_to :js, :html respond_to :js, :html
feature_category :issue_tracking
def index def index
@prioritized_labels = @available_labels.prioritized(@project) @prioritized_labels = @available_labels.prioritized(@project)
@labels = @available_labels.unprioritized(@project).page(params[:page]) @labels = @available_labels.unprioritized(@project).page(params[:page])
......
...@@ -7,6 +7,8 @@ module Projects ...@@ -7,6 +7,8 @@ module Projects
before_action :authorize_read_pod_logs! before_action :authorize_read_pod_logs!
before_action :ensure_deployments, only: %i(k8s elasticsearch) before_action :ensure_deployments, only: %i(k8s elasticsearch)
feature_category :logging
def index def index
if environment || cluster if environment || cluster
render :index render :index
......
...@@ -10,6 +10,8 @@ module EE ...@@ -10,6 +10,8 @@ module EE
include DescriptionDiffActions include DescriptionDiffActions
before_action :whitelist_query_limiting_ee, only: [:update] before_action :whitelist_query_limiting_ee, only: [:update]
feature_category :issue_tracking, [:delete_description_version, :description_diff]
end end
private private
......
...@@ -10,6 +10,8 @@ class Projects::InsightsController < Projects::ApplicationController ...@@ -10,6 +10,8 @@ class Projects::InsightsController < Projects::ApplicationController
track_unique_visits :show, target_id: 'p_analytics_insights' track_unique_visits :show, target_id: 'p_analytics_insights'
feature_category :insights
private private
def insights_entity def insights_entity
......
...@@ -17,6 +17,8 @@ module Projects ...@@ -17,6 +17,8 @@ module Projects
rescue_from ::Projects::Integrations::Jira::IssuesFinder::IntegrationError, with: :render_integration_error rescue_from ::Projects::Integrations::Jira::IssuesFinder::IntegrationError, with: :render_integration_error
rescue_from ::Projects::Integrations::Jira::IssuesFinder::RequestError, with: :render_request_error rescue_from ::Projects::Integrations::Jira::IssuesFinder::RequestError, with: :render_request_error
feature_category :integrations
def index def index
params[:state] = params[:state].presence || default_state params[:state] = params[:state].presence || default_state
......
...@@ -4,6 +4,8 @@ class Projects::Iterations::InheritedController < Projects::ApplicationControlle ...@@ -4,6 +4,8 @@ class Projects::Iterations::InheritedController < Projects::ApplicationControlle
before_action :check_iterations_available! before_action :check_iterations_available!
before_action :authorize_show_iteration! before_action :authorize_show_iteration!
feature_category :issue_tracking
def show; end def show; end
private private
......
...@@ -4,6 +4,8 @@ class Projects::IterationsController < Projects::ApplicationController ...@@ -4,6 +4,8 @@ class Projects::IterationsController < Projects::ApplicationController
before_action :check_iterations_available! before_action :check_iterations_available!
before_action :authorize_show_iteration! before_action :authorize_show_iteration!
feature_category :issue_tracking
def index; end def index; end
private private
......
...@@ -5,6 +5,8 @@ module Projects ...@@ -5,6 +5,8 @@ module Projects
before_action :authorize_read_licenses!, only: [:index] before_action :authorize_read_licenses!, only: [:index]
before_action :authorize_admin_software_license_policy!, only: [:create, :update] before_action :authorize_admin_software_license_policy!, only: [:create, :update]
feature_category :license_compliance
def index def index
respond_to do |format| respond_to do |format|
format.html do format.html do
......
...@@ -29,7 +29,8 @@ RSpec.describe "Every controller" do ...@@ -29,7 +29,8 @@ RSpec.describe "Every controller" do
next if controller.to_s.start_with?('P') && next if controller.to_s.start_with?('P') &&
!child_controller.to_s.start_with?('A', 'B', 'C', 'D', !child_controller.to_s.start_with?('A', 'B', 'C', 'D',
'E', 'F', 'G', 'H') 'E', 'F', 'G', 'H',
'I', 'J', 'K', 'L')
"#{controller}##{action}" "#{controller}##{action}"
end.compact end.compact
......
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