Logout link missing :method => 'delete'

[wallace]

The logout link, https://github.com/gregbell/active_admin/blob/master/lib/active_admin/views/header_renderer.rb#L30, is missing the :method parameter so a user gets a routing error for '/admin/logout'.

[mksys]

I get this too - is there an easy fix?

[wallace]

I thought I fixed it here #249 and I swear I had it work once or twice in a sample application but when I tested it in my application, it still didn't work.

Also, I realized I should've updated the specs to prove it worked. I'm going to keep working on it.

[oponder]

I think the issue is that active admin isn't loading the jquery rails file (called active_admin_vendor.js in active admin) which is what actually makes the :method => :delete work.

I forked activeadmin to fix this thinking there wasn't much development going on anymore before I realized that there were some really recent commits on the project.

Here is the commit that fixed it for me:
https://github.com/oponder/active_admin/commit/44d89e7180d4d657ddc57c93a7582f36c11042a4

[wallace]

Thanks @opondor, that explains it. I must've tested my fix in my real application when I still had jquery-rails installed (which I later removed).

I'll see if I can't find some time to create a clean fork using your commit and write a test for it too some time soon.

[libo]

While we wait to a proper fix to the problem I have this patch for /config/routes.rb

devise_scope :admin_user do
    get '/admin/logout', :to => 'active_admin/devise/sessions#destroy'
end

In this way we don't need to mess around with fork and so on...

[chetan-tomar]

Thanks @libo ...