Skip to content

Commit

Permalink
Use get/post instead of match on routes
Browse files Browse the repository at this point in the history
  • Loading branch information
arthurnn committed Oct 19, 2013
1 parent ef9dfa7 commit f732443
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions config/routes.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@
devise_for :users, :controllers => { :omniauth_callbacks => "users/omniauth_callbacks" }

# Hoptoad Notifier Routes
match '/notifier_api/v2/notices' => 'notices#create'
match '/locate/:id' => 'notices#locate', :as => :locate
match '/deploys.txt' => 'deploys#create'
post '/notifier_api/v2/notices' => 'notices#create'

This comment has been minimized.

Copy link
@mulder

mulder Nov 6, 2013

Does the javascript notification lib use post? I'm getting errors cause it is hitting this route with a GET.

This comment has been minimized.

Copy link
@soberstadt

soberstadt Dec 3, 2013

Contributor

it supports both. this should be changed back.

This comment has been minimized.

Copy link
@soberstadt

soberstadt Dec 3, 2013

Contributor

fixed with #617

get '/locate/:id' => 'notices#locate', :as => :locate
post '/deploys.txt' => 'deploys#create'

resources :notices, :only => [:show]
resources :deploys, :only => [:show]
Expand Down

0 comments on commit f732443

Please sign in to comment.