Merge branch 'dennis-enable-code-review-analytics-by-default' into 'master'

Enable Code Review Analytics by default

See merge request gitlab-org/gitlab!23285

changelogs/unreleased/dennis-enable-code-review-analytics-by-default.yml

+---
+title: Enable Code Review Analytics by default
+merge_request: 23285
+author:
+type: changed

ee/app/helpers/ee/projects_helper.rb

@@ -59,7 +59,7 @@ module EE
         nav_tabs << :packages
       end
 
-      if ::Feature.enabled?(:code_review_analytics, project)
+      if ::Feature.enabled?(:code_review_analytics, project, default_enabled: true)
         nav_tabs << :code_review
       end
 

ee/app/views/layouts/nav/sidebar/_project_analytics_link.html.haml

-- if Feature.enabled?(:code_review_analytics) && can?(current_user, :read_code_review_analytics, @project)
+- if Feature.enabled?(:code_review_analytics, default_enabled: true) && can?(current_user, :read_code_review_analytics, @project)
   - project_analytics_link = project_analytics_code_reviews_path(@project)
 
   = nav_link controller: :code_review do

ee/config/routes/project.rb

@@ -79,7 +79,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
         resources :audit_events, only: [:index]
 
         namespace :analytics do
-          constraints(::Constraints::FeatureConstrainer.new(:code_review_analytics)) do
+          constraints(::Constraints::FeatureConstrainer.new(:code_review_analytics, default_enabled: true)) do
             resources :code_reviews, only: [:index]
           end
         end

ee/lib/api/analytics/code_review_analytics.rb

@@ -29,7 +29,7 @@ module API
       end
 
       before do
-        not_found! unless Feature.enabled?(:code_review_analytics)
+        not_found! unless Feature.enabled?(:code_review_analytics, default_enabled: true)
       end
 
       resource :analytics do