Merge branch 'dz-scope-project-issues-routing' into 'master'

Copy issues routing under - scope

See merge request gitlab-org/gitlab!23779

changelogs/unreleased/dz-scope-project-issues-routing.yml

+---
+title: Copy issues routing under - scope
+merge_request: 23779
+author:
+type: changed

config/routes/issues.rb

+# frozen_string_literal: true
+
+get :issues, to: 'issues#calendar', constraints: lambda { |req| req.format == :ics }
+
+resources :issues, concerns: :awardable, constraints: { id: /\d+/ } do
+  member do
+    post :toggle_subscription
+    post :mark_as_spam
+    post :move
+    put :reorder
+    get :related_branches
+    get :can_create_branch
+    get :realtime_changes
+    post :create_merge_request
+    get :discussions, format: :json
+  end
+
+  collection do
+    post :bulk_update
+    post :import_csv
+  end
+end

config/routes/project.rb

@@ -401,25 +401,15 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
         end
       end
 
-      get :issues, to: 'issues#calendar', constraints: lambda { |req| req.format == :ics }
-
-      resources :issues, concerns: :awardable, constraints: { id: /\d+/ } do
-        member do
-          post :toggle_subscription
-          post :mark_as_spam
-          post :move
-          put :reorder
-          get :related_branches
-          get :can_create_branch
-          get :realtime_changes
-          post :create_merge_request
-          get :discussions, format: :json
-        end
+      # Unscoped route. It will be replaced with redirect to /-/issues/
+      # Issue https://gitlab.com/gitlab-org/gitlab/issues/118849
+      draw :issues
 
-        collection do
-          post :bulk_update
-          post :import_csv
-        end
+      # To ensure an old unscoped routing is used for the UI we need to
+      # add prefix 'as' to the scope routing and place it below original routing.
+      # Issue https://gitlab.com/gitlab-org/gitlab/issues/118849
+      scope '-', as: 'scoped' do
+        draw :issues
       end
 
       resources :notes, only: [:create, :destroy, :update], concerns: :awardable, constraints: { id: /\d+/ } do

ee/config/routes/issues.rb

+# frozen_string_literal: true
+
+resources :issues, only: [], constraints: { id: /\d+/ } do
+  member do
+    get '/descriptions/:version_id/diff', action: :description_diff, as: :description_diff
+    delete '/descriptions/:version_id', action: :delete_description_version, as: :delete_description_version
+    get '/designs(/*vueroute)', to: 'issues#designs', as: :designs, format: false
+  end
+
+  collection do
+    post :export_csv
+    get :service_desk
+  end
+
+  resources :issue_links, only: [:index, :create, :destroy], as: 'links', path: 'links'
+end

ee/config/routes/project.rb

@@ -123,21 +123,6 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
         end
       end
 
-      resources :issues, only: [], constraints: { id: /\d+/ } do
-        member do
-          get '/descriptions/:version_id/diff', action: :description_diff, as: :description_diff
-          delete '/descriptions/:version_id', action: :delete_description_version, as: :delete_description_version
-          get '/designs(/*vueroute)', to: 'issues#designs', as: :designs, format: false
-        end
-
-        collection do
-          post :export_csv
-          get :service_desk
-        end
-
-        resources :issue_links, only: [:index, :create, :destroy], as: 'links', path: 'links'
-      end
-
       get '/service_desk' => 'service_desk#show', as: :service_desk
       put '/service_desk' => 'service_desk#update', as: :service_desk_refresh