Commit fb49bdbb authored by Ash McKenzie's avatar Ash McKenzie

Merge branch 'dz-scope-notification-settings-route' into 'master'

Move notification_settings route under - scope

See merge request gitlab-org/gitlab!26983
parents a25a023f be477682
...@@ -133,6 +133,9 @@ Rails.application.routes.draw do ...@@ -133,6 +133,9 @@ Rails.application.routes.draw do
get :kill get :kill
end end
end end
# Notification settings
resources :notification_settings, only: [:create, :update]
end end
concern :clusterable do concern :clusterable do
...@@ -181,9 +184,6 @@ Rails.application.routes.draw do ...@@ -181,9 +184,6 @@ Rails.application.routes.draw do
# Spam reports # Spam reports
resources :abuse_reports, only: [:new, :create] resources :abuse_reports, only: [:new, :create]
# Notification settings
resources :notification_settings, only: [:create, :update]
resources :groups, only: [:index, :new, :create] do resources :groups, only: [:index, :new, :create] do
post :preview_markdown post :preview_markdown
end end
......
...@@ -69,7 +69,6 @@ Currently the following names are reserved as top level groups: ...@@ -69,7 +69,6 @@ Currently the following names are reserved as top level groups:
- `invites` - `invites`
- `jwt` - `jwt`
- `login` - `login`
- `notification_settings`
- `oauth` - `oauth`
- `profile` - `profile`
- `projects` - `projects`
......
...@@ -42,7 +42,6 @@ module Gitlab ...@@ -42,7 +42,6 @@ module Gitlab
invites invites
jwt jwt
login login
notification_settings
oauth oauth
profile profile
projects projects
......
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