Commit e9d57390 authored by Dmytro Zaporozhets's avatar Dmytro Zaporozhets

Use scoped profile routes by default

parent 1f42bb69
...@@ -180,6 +180,7 @@ Rails.application.routes.draw do ...@@ -180,6 +180,7 @@ Rails.application.routes.draw do
get 'jwks' => 'doorkeeper/openid_connect/discovery#keys' get 'jwks' => 'doorkeeper/openid_connect/discovery#keys'
draw :snippets draw :snippets
draw :profile
# Product analytics collector # Product analytics collector
match '/collector/i', to: ProductAnalytics::CollectorApp.new, via: :all match '/collector/i', to: ProductAnalytics::CollectorApp.new, via: :all
...@@ -266,7 +267,6 @@ Rails.application.routes.draw do ...@@ -266,7 +267,6 @@ Rails.application.routes.draw do
draw :uploads draw :uploads
draw :explore draw :explore
draw :admin draw :admin
draw :profile
draw :dashboard draw :dashboard
draw :user draw :user
draw :project draw :project
...@@ -274,13 +274,6 @@ Rails.application.routes.draw do ...@@ -274,13 +274,6 @@ Rails.application.routes.draw do
# Issue https://gitlab.com/gitlab-org/gitlab/-/issues/210024 # Issue https://gitlab.com/gitlab-org/gitlab/-/issues/210024
scope as: 'deprecated' do scope as: 'deprecated' do
draw :snippets draw :snippets
end
# Serve profile routes under /-/ scope.
# 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/210024
scope '-', as: :scoped do
draw :profile draw :profile
end end
......
...@@ -99,7 +99,7 @@ RSpec.describe DashboardOperationsProjectEntity do ...@@ -99,7 +99,7 @@ RSpec.describe DashboardOperationsProjectEntity do
let(:user) { build(:user, :admin) } let(:user) { build(:user, :admin) }
it 'shows the profile upgrade path' do it 'shows the profile upgrade path' do
expect(subject[:upgrade_path]).to eq '/profile/billings' expect(subject[:upgrade_path]).to eq '/-/profile/billings'
end end
end end
......
...@@ -33,7 +33,7 @@ RSpec.describe 'Profile > Account', :js do ...@@ -33,7 +33,7 @@ RSpec.describe 'Profile > Account', :js do
end end
it 'allows the user to disconnect when there is an existing identity' do it 'allows the user to disconnect when there is an existing identity' do
expect(page).to have_link('Disconnect Twitter', href: '/profile/account/unlink?provider=twitter') expect(page).to have_link('Disconnect Twitter', href: '/-/profile/account/unlink?provider=twitter')
end end
it 'shows active for a provider that is not allowed to unlink' do it 'shows active for a provider that is not allowed to unlink' do
......
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